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

#164046267 Users can highlight text and comment on it #36

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

cob04
Copy link
Contributor

@cob04 cob04 commented Mar 20, 2019

Description

This feature gives the ability to highlight specific text in an article and comment on it


**Type of change**
  • new feature(non-breaking change which adds functionality).

How has this been tested

  • Unit tests

How to test this manually

  • Clone the repository https://github.com/andela/ah-legion-backend.git
  • run git checkout --track origin/ft-article-hightlight-comment-164046267

After logging in and creating an article;
Test the following endpoints using postman:

POST /api/articles/<slug>/comments/ - Create an article comment
PATCH /api/articles/<slug>/comments/<pk>/ - Update a comment
POST /api/articles/<slug>/comments/<pk>/ - Create a comment for a another comment

Checklist:

  • This follows all the guidelines for this project.

Related Stories

#164046267

@BrianSerem BrianSerem temporarily deployed to ah-legion-staging-pr-36 March 20, 2019 12:15 Inactive
- add highlight field to comments model
- add is_highlighted field to comment model
- add highlight field to comment sereliazers

[finishes #164046267]
Copy link

@RachelleMaina RachelleMaina left a comment

Choose a reason for hiding this comment

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

Providing an example request body in the PR message would go along way in easing the review. Thank you.

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.

None yet

4 participants