-
Notifications
You must be signed in to change notification settings - Fork 344
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
Add Following tab to the user profile view #1463
Conversation
Implemented the suggested changes in 2 commits. @brylie Could you please have a look? |
@brylie I renamed the view accordingly to the comments above. |
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.
Overall, this PR looks good. There is just a bit more ambiguity in how we use "following" and "followers" here that we need to iron out.
Code Climate has analyzed commit 7396104 and detected 0 issues on this pull request. View more on Code Climate. |
I believe everything is ironed out now. @brylie asking again for review. |
@all-contributors please add @wippo-devops for code |
I've put up a pull request to add @wippo-devops! 🎉 |
Closes #1447
All of these tasks should be done in the accounts app.
Description
Created a new view that shows followers of user (/profile//following). View shows clicable image icons of profile pictures of followers. If there are zero followers it shows message that no one is following.
I was thinking it might be good idea to add this to profile below the "About me" section, but it was not mentioned in the issue.