Skip to content

feat(users): add follow/unfollow API #1333

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 28, 2021
Merged

feat(users): add follow/unfollow API #1333

merged 1 commit into from
Feb 28, 2021

Conversation

max-wittig
Copy link
Member

No description provided.

@max-wittig max-wittig self-assigned this Feb 25, 2021
@max-wittig max-wittig marked this pull request as draft February 25, 2021 21:39
max-wittig
max-wittig commented Feb 26, 2021
View reviewed changes
@max-wittig max-wittig force-pushed the feat/user-follow-api branch 2 times, most recently from d6aec3a to cf585b6 Compare February 26, 2021 21:14
max-wittig
max-wittig commented Feb 26, 2021
View reviewed changes
@max-wittig max-wittig marked this pull request as ready for review February 26, 2021 21:15
@max-wittig max-wittig requested a review from nejch February 26, 2021 21:15
@max-wittig max-wittig assigned nejch and unassigned max-wittig Feb 26, 2021
nejch
nejch requested changes Feb 27, 2021
View reviewed changes
Copy link
Member

@nejch nejch left a comment

Choose a reason for hiding this comment

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

Cool! It all looks good to me, just that little docstring return (unless you want to return a User, see other thread).

@nejch nejch assigned max-wittig and unassigned nejch Feb 27, 2021
@max-wittig max-wittig assigned nejch and unassigned max-wittig Feb 27, 2021
@nejch nejch merged commit 5bc158d into master Feb 28, 2021
@nejch nejch deleted the feat/user-follow-api branch May 30, 2021 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nejch nejch nejch approved these changes

Assignees

@nejch nejch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@max-wittig @nejch