Skip to content

Commit

Permalink
Returning project owner & supporter names rather than IDs in serializer
Browse files Browse the repository at this point in the history
  • Loading branch information
rosiejeays committed Sep 5, 2023
1 parent 3db0483 commit 4f139f0
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 2 deletions.
2 changes: 1 addition & 1 deletion crowdfunding/projects/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ class Project(models.Model):
owner = models.ForeignKey(
get_user_model(),
on_delete=models.CASCADE,
related_name='owned_projects'
related_name='owner_projects'
)
is_deleted = models.BooleanField(default=False)

Expand Down
2 changes: 2 additions & 0 deletions crowdfunding/projects/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,14 @@

class PledgeSerializer(serializers.ModelSerializer):
supporter = serializers.ReadOnlyField(source='supporter.id')
supporter = serializers.StringRelatedField()
class Meta:
model = apps.get_model('projects.Pledge')
fields = '__all__'

class ProjectSerializer(serializers.ModelSerializer):
owner = serializers.ReadOnlyField(source='owner.id')
owner = serializers.StringRelatedField()
class Meta:
model = apps.get_model('projects.Project')
fields = '__all__'
Expand Down
9 changes: 8 additions & 1 deletion crowdfunding/users/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,11 @@ class CustomUser(AbstractUser):
pass

def __str__(self):
return self.username
"""
Return the first_name plus the last_name, with a space in between.
"""
if AbstractUser.get_full_name(self) != '':
name = AbstractUser.get_full_name(self)
else:
name = self.username
return name

0 comments on commit 4f139f0

Please sign in to comment.