-
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
#167224203 get highlighted text #75
Conversation
3d5896c
to
54ab1b9
Compare
f6679c8
to
7f5939c
Compare
@@ -63,6 +63,9 @@ articleRouter.post('/:slug/highlight', | |||
validateInputs('highlight', highlightFields), | |||
articleController.highlight); | |||
|
|||
articleRouter.get('/:slug/highlight', checkArticle.getArticle, articleController.getHighlight); | |||
|
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.
please remove these empty lines @alainburindi
7f5939c
to
37207f4
Compare
@@ -49,5 +49,6 @@ export default { | |||
parentCommentId: 'You can not reply to a reply', | |||
notCommentOwner: 'You must be the owner of the comment in order to proceed', | |||
noBookmark: 'You have no bookmarks', | |||
noSlug: 'There is no article with the slug provided' | |||
noSlug: 'There is no article with the slug provided', | |||
noHighlight: 'No highlited text for this article', |
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.
@alainburindi it's highlighted
not highlited
This mistake is happening across several files so please correct in other places, as well.
37207f4
to
5d4e312
Compare
What does this PR do?
Have the
get highlighted text
workingDescription of Task to be completed?
Have the following endpoints working
/api/articles/:slug/highlight
: highlight a textHow should this be manually tested?
npm install
npm run dev
What are the relevant pivotal tracker stories?
#167224203