-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#165890222 following and unfollowing users #36
#165890222 following and unfollowing users #36
Conversation
300b62e
to
d39817f
Compare
d39817f
to
cac131f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested the pull request on Heroku and it works
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
profile = UserProfile.objects.get(user=value.follower) | ||
profile = FetchUserProfileSerializer(profile) | ||
profiles.append(profile.data) | ||
return Response(profiles, status=status.HTTP_200_OK,) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joelmugaya We should return a message You have no followers
when the list being returned empty.
profile = UserProfile.objects.get(user=value.followed) | ||
profile = FetchUserProfileSerializer(profile) | ||
profiles.append(profile.data) | ||
return Response(profiles, status=status.HTTP_200_OK,) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joelmugaya We should return a message You are following anyone.
when the list being returned empty.
cac131f
to
9daafe3
Compare
9daafe3
to
d05d3f2
Compare
d05d3f2
to
a948a6d
Compare
a948a6d
to
eabd9da
Compare
- update profile model to include following functionality - update profile serializer to add a following field to the profile serialized data - write tests for the following functionality - add endpoints for the follow functionality to the url.py file - write functions to handle the follow functionlity [finishes #165890222]
eabd9da
to
3dcd6d6
Compare
What does the PR do?
Description of Task to be completed?
Tasks to be completed
How should this be manually tested?
POST /api/profiles/<username>/follow/
returns"You are following username."
GET /api/users/profile/<username>/
returnsfollowing: True
DELETE /api/profiles/<username>/follow/
GET /api/profiles/<username>/followers/
GET /api/profiles/<username>/following/
[finishes #165890222]