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

new: delete bookmark using "postId" #225

Merged
merged 2 commits into from
Aug 7, 2023
Merged

Conversation

tobisupreme
Copy link
Contributor

This PR...

modifies the API endpoint to delete individual bookmarks. To delete a bookmark, pass the id of the bookmarked content as a query parameter to the API:

DELETE /bookmarks?postId=id-of-bookmarked-content

Checklist

  • tested locally
  • added new dependencies
  • updated the Swagger API docs
  • updated the Postman collection
  • added a test
  • linter passes
  • format check passes

Fixes #224

This commit modifies the API endpoint to delete individual bookmarks
To delete a bookmark, pass the id of the bookmarked content as a query parameter to the API
@tobisupreme tobisupreme requested a review from Boye95 August 7, 2023 19:24
@tobisupreme tobisupreme linked an issue Aug 7, 2023 that may be closed by this pull request
@tobisupreme tobisupreme temporarily deployed to Pre-deployment August 7, 2023 19:24 — with GitHub Actions Inactive
Copy link
Contributor

@Boye95 Boye95 left a comment

Choose a reason for hiding this comment

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

Good job 🎉

@Boye95 Boye95 merged commit 7e988e0 into dev Aug 7, 2023
1 check passed
@tobisupreme tobisupreme deleted the remove-bookmarks-without-bookmarkid branch August 7, 2023 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove bookmarks without "bookmarkId"
2 participants