Skip to content

Implement social followers endpoints#30

Draft
Weissnix4711 wants to merge 1 commit intometabrainz:masterfrom
Weissnix4711:social
Draft

Implement social followers endpoints#30
Weissnix4711 wants to merge 1 commit intometabrainz:masterfrom
Weissnix4711:social

Conversation

@Weissnix4711
Copy link

/1/user/(user_name)/followers
/1/user/(user_name)/following
/1/user/(user_name)/follow
/1/user/(user_name)/unfollow

Untested code

@georgkrause
Copy link
Collaborator

@Weissnix4711 Thank you, this looks like a useful contribution! It seems there is a minor typo, which causes the tests to fail.

However, I think it would be good to have some basic unit testing around the new functions, would you like to add them? I can assist. If you are not interested in testing, I can take the task! Let me know :)

@Weissnix4711
Copy link
Author

Feel free to take over this PR if you want. I'll admit this is very hastily thrown together and I'm currently away and unable to really work on anything. But I figured if I don't open a PR now I will probably forget lol.

@georgkrause georgkrause self-assigned this Feb 2, 2024
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.

2 participants