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.

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

[Backend] Followed Categories Endpoint #442

Merged
merged 2 commits into from
Dec 17, 2022
Merged

[Backend] Followed Categories Endpoint #442

merged 2 commits into from
Dec 17, 2022

Conversation

oguzhandemirelx
Copy link
Contributor

Description:

In order to check whether a user followed a category or not, we need to implement an endpoint. Mobil and frontend teams can use this endpoint to show followed categories.

Todo's:

  • New endpoint for followed categories.
  • Add unit tests for that endpoint.

Notes:

  • GET profile/followed_categories endpoint added and tested.

Reviewer: @mehmetemreakbulut

Task Deadline: 17.12.2022 22:00
Review Deadline: 17.12.2022 22:00

@oguzhandemirelx oguzhandemirelx added Prio: Medium Importance of Issue: Medium Status: Need Review Issue work is done, waiting to be reviewed Team: Backend Backend based work - for backend team labels Dec 17, 2022
@oguzhandemirelx oguzhandemirelx self-assigned this Dec 17, 2022
mehmetemreakbulut
mehmetemreakbulut approved these changes Dec 17, 2022
View reviewed changes
Copy link
Contributor

@mehmetemreakbulut mehmetemreakbulut left a comment

Choose a reason for hiding this comment

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

URL is set and test is written. LGTM!

@mehmetemreakbulut mehmetemreakbulut merged commit e7c0e8f into master Dec 17, 2022
@oguzhandemirelx oguzhandemirelx added Status: Accepted Issue is reviewed, accepted and removed Status: Need Review Issue work is done, waiting to be reviewed labels Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mehmetemreakbulut mehmetemreakbulut mehmetemreakbulut approved these changes

Assignees

@oguzhandemirelx oguzhandemirelx

Labels
Prio: Medium Importance of Issue: Medium Status: Accepted Issue is reviewed, accepted Team: Backend Backend based work - for backend team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oguzhandemirelx @mehmetemreakbulut