Skip to content
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

#164046256 Users Should Follow Each Other #22

Merged
merged 1 commit into from
Mar 15, 2019

Conversation

BrianSerem
Copy link
Contributor

@BrianSerem BrianSerem commented Mar 13, 2019

Description

This Pull request implements users' abiity to follow each other, an existing user may be followed by another user.
These endpoints will now be working:

GET /api/profiles/<username>/follow/ - Follow a user
DELETE /api/profiles/<username>/follow/ - Unfollow a user
GET /api/profiles/<username>/following/ - Get a users statistics: followers/followees


**Type of change**
  • new feature(non-breaking change which adds functionality).

How has this been tested

  • End to End
  • Integration

How to test this manually
Using Postman, navigate to https://ah-legion-staging-pr-22.herokuapp.com, use instructions below to test the various endpoints.

: Follow a user

https://ah-legion-staging-pr-22.herokuapp.com/api/profiles/

Unfollow a user:

https://ah-legion-staging-pr-22.herokuapp.com/api/profiles/username/

Get user statistics: Number of followers and followees:

https://ah-legion-staging-pr-22.herokuapp.com/api/profiles/username/followings/


**Checklist:**
  • This follows all the guidelines for this project.

Related Stories


#164046256

@BrianSerem BrianSerem temporarily deployed to ah-legion-staging-pr-22 March 13, 2019 14:50 Inactive
@BrianSerem BrianSerem temporarily deployed to ah-legion-staging-pr-22 March 13, 2019 16:19 Inactive
@BrianSerem BrianSerem force-pushed the ft-users-follow-eachother-164046256 branch from f5cd7b9 to dd22308 Compare March 14, 2019 05:03
@BrianSerem BrianSerem temporarily deployed to ah-legion-staging-pr-22 March 14, 2019 05:03 Inactive
Copy link
Contributor

@cob04 cob04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Copy link
Contributor

@ElMonstro ElMonstro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

authors/apps/authentication/tests/test_social_auth.py Outdated Show resolved Hide resolved
authors/settings/base.py Outdated Show resolved Hide resolved
@BrianSerem BrianSerem force-pushed the ft-users-follow-eachother-164046256 branch from dd22308 to e4278d6 Compare March 14, 2019 13:08
@BrianSerem BrianSerem requested a deployment to ah-legion-staging-pr-22 March 14, 2019 13:09 Abandoned
@BrianSerem BrianSerem force-pushed the ft-users-follow-eachother-164046256 branch from e4278d6 to d697b31 Compare March 14, 2019 13:13
@BrianSerem BrianSerem temporarily deployed to ah-legion-staging-pr-22 March 14, 2019 13:13 Inactive
@BrianSerem BrianSerem force-pushed the ft-users-follow-eachother-164046256 branch from d697b31 to 515269d Compare March 14, 2019 13:35
@BrianSerem BrianSerem temporarily deployed to ah-legion-staging-pr-22 March 14, 2019 13:35 Inactive
@BrianSerem BrianSerem force-pushed the ft-users-follow-eachother-164046256 branch from 515269d to ee2e9ac Compare March 14, 2019 14:17
@BrianSerem BrianSerem temporarily deployed to ah-legion-staging-pr-22 March 14, 2019 14:18 Inactive
Copy link
Contributor

@abulojoshua1 abulojoshua1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BrianSerem, Please fix the failing build.

-write tests on following/follower feature
-create manytomany field on follower/followee
-create views and serializers on this feature
-verify if tests running

[Finishes #164046256]
@BrianSerem BrianSerem force-pushed the ft-users-follow-eachother-164046256 branch from ee2e9ac to 7ce5218 Compare March 15, 2019 05:14
@BrianSerem
Copy link
Contributor Author

@BrianSerem, Please fix the failing build.

Was waiting for @ElMonstro 's PR to be merged, it fixed the failing build

Copy link
Contributor

@abulojoshua1 abulojoshua1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@abulojoshua1 abulojoshua1 merged commit a6cfa89 into develop Mar 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants