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

#159952011 Favorite and Unfavorite articles(Articles) #27

Merged
merged 1 commit into from
Sep 23, 2018

Conversation

Estaer
Copy link
Contributor

@Estaer Estaer commented Sep 19, 2018

What does this PR do?

  • Favorite and unfavorite articles

Description of Tasks to be completed?
Have the following endpoints working

POST /api/articles/<article_slug>/favorite
DELETE /api/articles/<article_slug>/unfavorite

How should this be manually tested?

  • After cloning the repo, cd into the directory.
  • Run pip install -r requirements.txt
  • Use Postman to try out the endpoints.
  • Add an authorization header with the token of the logged in user.

Only authenticated users are allowed to perform this action.

@Estaer Estaer had a problem deploying to ah-backend-staging-pr-27 September 19, 2018 15:29 Failure
@coveralls
Copy link

coveralls commented Sep 19, 2018

Pull Request Test Coverage Report for Build 635

  • 56 of 56 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.09%) to 98.496%

Totals Coverage Status
Change from base Build 539: 0.09%
Covered Lines: 917
Relevant Lines: 931

💛 - Coveralls

@Estaer Estaer force-pushed the feat_favorite_unfavorite_articles_159952011 branch from 3c47f08 to 21c2782 Compare September 19, 2018 15:46
@Estaer Estaer had a problem deploying to ah-backend-staging-pr-27 September 19, 2018 15:46 Failure
@Estaer Estaer force-pushed the feat_favorite_unfavorite_articles_159952011 branch 4 times, most recently from 2e359a4 to d08c380 Compare September 21, 2018 16:23
@Estaer Estaer force-pushed the feat_favorite_unfavorite_articles_159952011 branch from d08c380 to 1bb2265 Compare September 21, 2018 16:35
@inno-asiimwe inno-asiimwe merged commit e6d51e7 into develop Sep 23, 2018
@inno-asiimwe inno-asiimwe deleted the feat_favorite_unfavorite_articles_159952011 branch September 23, 2018 18:23
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

3 participants