-
Notifications
You must be signed in to change notification settings - Fork 8
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
#165429968 User can get their bookmarked articles #108
#165429968 User can get their bookmarked articles #108
Conversation
- create route - write test - add middleware - create controller - add documentation [Finishes 165429968]
- fix documentation [Finishes 165429968]
const bookmarkController = { toggleBookmark }; | ||
const userGetTheirBookmarkArticles = async (req, res) => { | ||
try { | ||
const bookmarkedArticle = await Bookmark.findAll({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be bookmarkedArticles @AnayoOleru
- edit variable name
…into feature/165429968-user-get-bookmark-articles
3cf8368
to
daaa14f
Compare
@@ -62,5 +62,43 @@ const toggleBookmark = async (req, res) => { | |||
} | |||
}; | |||
|
|||
const bookmarkController = { toggleBookmark }; | |||
const userGetTheirBookmarkArticles = async (req, res) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can name this getBookMarkedArticlesForUser
. userGetTheirBookmarkArticles
is just awkward @AnayoOleru
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can name this
getBookMarkedArticlesForUser
.userGetTheirBookmarkArticles
is just awkward @AnayoOleru
That's true. I've fixed it
- edit function name
- refactor documentation
…into feature/165429968-user-get-bookmark-articles
Description
Currently, users can bookmark articles, but they cannot get the articles they bookmarked. A user bookmarks articles to read them later or save them, but without the ability to get these bookmarked articles, they'll be saving articles for nothing. This P.R, adds the ability for a user to be able to get his/her articles they'd bookmarked.
Type of change
How Has This Been Tested?
Checklist:
PT-ID
#165429968
Screenshots
Questions:
N/A