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

Annotations #639

Merged
merged 13 commits into from
Dec 19, 2023
Merged

Annotations #639

merged 13 commits into from
Dec 19, 2023

Conversation

Simurgan
Copy link
Contributor

Annotations are implemented on backend. This PR closes #603

Bera0422 and others added 12 commits December 15, 2023 09:55
@Bera0422
Create annotation project
73264b0 
Created a new Django project for annotations. TextSelector annotation model is implemented.
@Simurgan
create models
022ea92 
Co-authored-by: Ahmed Bera Pay abera.pay@gmail.com
@Simurgan @Bera0422
fix settings.py and add on_delete to foreign keys in models.
694d840 
Co-authored-by: Ahmed Bera Pay <abera.pay@gmail.com>
@Simurgan @Bera0422
remove unnecessary apps.
b1204cc 
Co-authored-by: Ahmed Bera Pay <abera.pay@gmail.com>
@Simurgan
remove forgotten django rest framework methods and classes
09f4631
@Bera0422
Merge pull request #615 from bounswe/annotations-post-api
ba9e45c 
remove forgotten django rest framework methods and classes
@Bera0422
Implement GET API
60808a1 
Implemented GET methods to
- return matched annotations according to source URI and creator email
- return single annotation given its ID
@Bera0422
Update views.py
51e1900 
Modified to get scheme dynamically
@Simurgan
Merge pull request #616 from bounswe/annotations-get
1ee771b 
Implement GET API
@Simurgan
implement annotations post api
a20bdd4
@Simurgan
implement annotations post api tests
193b32a
@Bera0422
Merge pull request #636 from bounswe/annotations-post-api
05e3cda 
Annotations post api
@Simurgan Simurgan added priority: critical This issue requires immediate attention to prevent major issues or disruptions. size: large Requires significant effort and may take several weeks to months to complete. difficulty: challenging Requires advanced skills, knowledge, and may involve complex problems or significant research/collab type: implementation This issue is related to implementation backend issues related to backend labels Dec 19, 2023
@Simurgan Simurgan linked an issue Dec 19, 2023 that may be closed by this pull request
BE - Annotations Implementation #603
Closed
4 tasks
Bera0422
Bera0422 approved these changes Dec 19, 2023
View reviewed changes
Copy link
Contributor

@Bera0422 Bera0422 left a comment

Choose a reason for hiding this comment

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

Annotation retrieving and creation implementation is compatible with W3C standards. I think it is OK to merge.

@hakanaktas0 hakanaktas0 merged commit 62ab787 into backend Dec 19, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bera0422 Bera0422 Bera0422 approved these changes

@hakanaktas0 hakanaktas0 Awaiting requested review from hakanaktas0

Assignees

@Simurgan Simurgan

@Bera0422 Bera0422

Labels
backend issues related to backend difficulty: challenging Requires advanced skills, knowledge, and may involve complex problems or significant research/collab priority: critical This issue requires immediate attention to prevent major issues or disruptions. size: large Requires significant effort and may take several weeks to months to complete. type: implementation This issue is related to implementation
Projects
None yet
Milestone
CMPE451 Final Milestone: Presentation...
Development

Successfully merging this pull request may close these issues.

BE - Annotations Implementation
3 participants
@Simurgan @Bera0422 @hakanaktas0