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

Add Notifications collection and add/modify APIs to add notification view functionality #322

Draft
wants to merge 38 commits into
base: main
Choose a base branch
from
Draft

Add Notifications collection and add/modify APIs to add notification view functionality #322

wants to merge 38 commits into from

Conversation

Sebastian-Webster
Copy link
Member

@Sebastian-Webster Sebastian-Webster commented Nov 24, 2023 •
edited
Loading

Schemas to be created:

  1. Notification - Done in e658c58

New APIs to be created:

  1. POST temp/getnotifications - Done
  2. POST temp/deletenotification (takes notificationId in body) - Done in 2117125
  3. POST temp/clearnotifications - Done in 55f04f8

APIs to be modified:

  1. temp/deleteimage
  2. temp/deletepoll
  3. temp/deletethread
  4. temp/deleteaccount
  5. temp/followuser
  6. temp/unfollowuser
  7. temp/deletecomment
  8. temp/voteoncomment
  9. temp/voteonpost
  10. temp/removevoteoncomment
  11. temp/removevoteonpost
  12. temp/blockaccount
  13. temp/postcomment

@Sebastian-Webster Sebastian-Webster added the enhancement New feature or request label Nov 24, 2023
@Sebastian-Webster Sebastian-Webster self-assigned this Nov 24, 2023
@Sebastian-Webster Sebastian-Webster linked an issue Nov 24, 2023 that may be closed by this pull request
Add Notification collection and APIs to retrieve and delete Notifications #321
Open
@Sebastian-Webster Sebastian-Webster marked this pull request as draft November 24, 2023 13:38
Sebastian-Webster and others added 19 commits November 25, 2023 16:26
@Sebastian-Webster
Change userId test selection process
8048984
@Sebastian-Webster
Added temp/deletenotification API
2117125
@Sebastian-Webster
Added first temp/deletenotification test
91ed4b7
@Sebastian-Webster
Added temp/deletenotification type tests
7689bce
@Sebastian-Webster
Added temp/deletenotification not found tests
cce588c
@Sebastian-Webster
Fixed bug in tests
e55cf34
@Sebastian-Webster
Fixed bug in tests
28f639a
@Sebastian-Webster
Fixed bug in tests
4ea682a
@Sebastian-Webster
Fixed bug in tests
28c69dd
@Sebastian-Webster
Added skeleton getnotifications API
175092c
@Sebastian-Webster
Added more to temp/getnotifications
f52ac28
@Sebastian-Webster
Finished temp/getnotifications API (still need tests)
0dfd4fd
@Sebastian-Webster
Fixed server crashing issue
7b3439b
@Sebastian-Webster
Merge branch 'main' into 321-add-notification-collection-and-apis-to-…
f8a318d 
…retrieve-and-delete-notifications
@Sebastian-Webster
Stringify notification id
5a8742f
@Sebastian-Webster
Add lean option to temp/getnotifications
e8a2a24
@Sebastian-Webster
Add getnotifications test skeleton file
e000918
@Sebastian-Webster
Partially complete one delete notification test (not done yet)
7f71777
@Sebastian-Webster
Added comment deletion fails if not comment owner test
bb096db
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Sebastian-Webster Sebastian-Webster

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Notification collection and APIs to retrieve and delete Notifications
1 participant
@Sebastian-Webster