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

[Backend] Report Endpoint Implementation #503

Merged
merged 1 commit into from
Dec 26, 2022

Conversation

canberkboun9
Copy link
Contributor

Description:

Task 4 in Meeting 24.1

  • Report functionality will have POST requests. Reports will be seen in admin panel by Admins.
  • Endpoint is implemented
  • Postman tests were done.
  • One unit test was written.

Tasks Done:

  • report_content implementation
  • Unit tests
  • Documentation
  • Manual tests

Reviewer:

Notes:

  • I will write more unit tests tomorrow. Thus, review may wait.

Issue:

@canberkboun9 canberkboun9 added Prio: Medium Importance of Issue: Medium Type: Enhancement New features via code and/or design. Team: Backend Backend based work - for backend team labels Dec 25, 2022
@canberkboun9 canberkboun9 self-assigned this Dec 25, 2022
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.

LGTM!

@mehmetemreakbulut mehmetemreakbulut merged commit c5ded17 into master Dec 26, 2022
@canberkboun9 canberkboun9 mentioned this pull request Dec 26, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prio: Medium Importance of Issue: Medium Team: Backend Backend based work - for backend team Type: Enhancement New features via code and/or design.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants