-
Notifications
You must be signed in to change notification settings - Fork 1
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
#161254665 Add comments to articles #32
Conversation
ab8bba0
to
37cbb81
Compare
37cbb81
to
226feff
Compare
0dd1576
to
d2a9df2
Compare
Pull Request Test Coverage Report for Build 575
💛 - Coveralls |
bc39b17
to
849f626
Compare
849f626
to
abf92a0
Compare
Add tests to improve the coverage |
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.
Add tests to improve the coverage
solve the merging conflict
abf92a0
to
63f34ce
Compare
- enable users to add comments to articles - enable users to comment on other comments - add tests for comments - fix failing travis build - fix hound issues - fix codeclimate issues [Delivers #161254665]
63f34ce
to
9552411
Compare
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.
LGTM
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.
LGTM
What does this PR do?
This PR merges the add comments feature branch to develop
Description of Task to be completed?
- Allow users to comment on articles
- Allow users to view all comments of an article
- Allow users to delete comments on articles
- Allow users to comment on other comments
How should this be manually tested?
Any background context you want to provide?
Comments allow authors to get feedback on their articles.
What are the relevant pivotal tracker stories?
#161254665
Screenshots