fix(article): use id instead of created_at for order_by

sparcs-kaist · Jan 23, 2024 · b05dbda · b05dbda
1 parent 06fc587
commit b05dbda
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/apps/core/migrations/0057_alter_article_name_type_and_more.py b/apps/core/migrations/0057_alter_article_name_type_and_more.py
@@ -23,8 +23,7 @@ class Migration(migrations.Migration):
         migrations.AddIndex(
             model_name="article",
             index=models.Index(
-                fields=["created_at", "parent_board_id"],
-                name="created_at_parent_board_id_idx",
+                fields=["id", "parent_board_id"], name="id_parent_board_id_idx"
             ),
         ),
     ]
diff --git a/apps/core/models/article.py b/apps/core/models/article.py
@@ -160,8 +160,8 @@ class Meta(MetaDataModel.Meta):
 
         indexes = [
             models.Index(
-                fields=["created_at", "parent_board_id"],
-                name="created_at_parent_board_id_idx",
+                fields=["id", "parent_board_id"],
+                name="id_parent_board_id_idx",
             )
         ]
 

diff --git a/apps/core/views/viewsets/article.py b/apps/core/views/viewsets/article.py
@@ -45,7 +45,7 @@ class ArticleViewSet(viewsets.ModelViewSet, ActionAPIViewSet):
     queryset = Article.objects.all()
 
     filterset_class = ArticleFilter
-    ordering_fields = ["created_at", "positive_vote_count"]
+    ordering_fields = ["id", "positive_vote_count"]
 
     serializer_class = ArticleSerializer
     action_serializer_class = {