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

Added annotation crud endpoints #1025

Merged
merged 2 commits into from
Dec 17, 2023
Merged

Added annotation crud endpoints #1025

merged 2 commits into from
Dec 17, 2023

Conversation

halisbal
Copy link
Contributor

I implemented the update, delete and get endpoints for annotation model. Also changed the body field type in the annotation model, updated that as a list.

@halisbal halisbal requested a review from UnalDeniz December 17, 2023 16:03
@halisbal halisbal self-assigned this Dec 17, 2023
@halisbal halisbal added Priority: Medium Issues that are important but not critical, and can be worked on after higher priority issues. Type: Enhancement Issues that suggest improvements to existing features or functionality. Status: In Review Issues that are waiting for review and approval from others. Team: Backend Issues related to backend team Type: Feature Issues related to adding new features or functionality to the project. and removed Type: Enhancement Issues that suggest improvements to existing features or functionality. labels Dec 17, 2023
UnalDeniz
UnalDeniz approved these changes Dec 17, 2023
View reviewed changes
Copy link
Contributor

@UnalDeniz UnalDeniz left a comment

Choose a reason for hiding this comment

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

I have reviewed your code and tested it. Everything works as intended. Good job.

@UnalDeniz UnalDeniz merged commit 4bca5de into main Dec 17, 2023
@UnalDeniz UnalDeniz added Status: Done Issues that have been completed. and removed Status: In Review Issues that are waiting for review and approval from others. labels Dec 17, 2023
@halisbal halisbal mentioned this pull request Dec 19, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@UnalDeniz UnalDeniz UnalDeniz approved these changes

Assignees

@halisbal halisbal

Labels
Priority: Medium Issues that are important but not critical, and can be worked on after higher priority issues. Status: Done Issues that have been completed. Team: Backend Issues related to backend team Type: Feature Issues related to adding new features or functionality to the project.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@halisbal @UnalDeniz @haliskunduz