small fix

Author: XiyuanWu

cards/serializers.py

@@ -44,7 +44,7 @@ def get_card_name(self, obj):
 
     class Meta:
         model = UserCard
-        fields = ("id", "card", "card_id", "card_name", "user", "notes", "is_active")
+        fields = ("id", "card", "card_name", "user", "notes", "is_active")
         read_only_fields = ()
 
         validators = [

optimizer/models.py

@@ -3,7 +3,7 @@
 from cards.models import RewardRule
 
 # Create your models here.
-# This function get card user have(make api call for http://127.0.0.1:8000/api/cards/user-cards/)
+# Model to store the user's category selections for the optimizer
 class UserCategorySelection(models.Model):
     user = models.ForeignKey(settings.AUTH_USER_MODEL, on_delete=models.CASCADE, related_name="optimizer_selections")
     category_tag = models.CharField(max_length=255, choices=RewardRule.CATEGORY_CHOICES)

optimizer/serializers.py

@@ -10,7 +10,6 @@ class Meta:
         read_only_fields = ("user",)
 
     def create(self, validated_data):
-        validated_data['user'] = self.context['request'].user
         try:
             return super().create(validated_data)
         except IntegrityError:

optimizer/services.py

@@ -4,7 +4,7 @@
 
 # Most of code logic are generated by AI, but with human fix the code and correct logic.
 # eg, at first the code is calculate the best card for a category from card database,
-# but it should be calculate the best card for a category from user's cards.
+# but it should calculate the best card for a category from user's cards.
 
 SELECTED = "SELECTED_CATEGORIES"
 BASE_ANYWHERE = "OTHER"