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

#166840975 user can rate artilce #13

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

Conversation

Alpha1202
Copy link
Contributor

What does this PR do?

  • An authenticated user can rate an article

Description of Task to be completed?

  • add a temporary login form
  • add a temporary get all articles page
  • add read one article page
  • add a dropdown button
  • add the star rating modal
  • add the on-success notification

How should this be manually tested?

  • log in to the app to get a token

  • Click on any article to read

  • Click on the dropdown button to view the rate article link

Any background context you want to provide?

  • User should be to give ratings to articles

What are the relevant pivotal tracker stories?

Screenshots (if appropriate)

Screenshot 2019-08-07 at 3 54 01 AM

Questions:

  • N/A

@Cavdy Cavdy temporarily deployed to ah-nyati-frontend-pr-13 August 7, 2019 02:58 Inactive
@Alpha1202 Alpha1202 force-pushed the ft-rate-articles-166840975 branch from c488b6c to 937fa67 Compare August 29, 2019 15:58
@Cavdy Cavdy temporarily deployed to ah-nyati-frontend-pr-13 August 29, 2019 15:58 Inactive
@Alpha1202 Alpha1202 force-pushed the ft-rate-articles-166840975 branch from 937fa67 to 3dbf05d Compare August 29, 2019 16:38
@Cavdy Cavdy temporarily deployed to ah-nyati-frontend-pr-13 August 29, 2019 16:38 Inactive
@Alpha1202 Alpha1202 force-pushed the ft-rate-articles-166840975 branch from 3dbf05d to 4b392b7 Compare August 29, 2019 16:42
@Cavdy Cavdy temporarily deployed to ah-nyati-frontend-pr-13 August 29, 2019 16:43 Inactive
@Alpha1202 Alpha1202 force-pushed the ft-rate-articles-166840975 branch from 4b392b7 to ae7a9dc Compare September 2, 2019 10:04
@Cavdy Cavdy temporarily deployed to ah-nyati-frontend-pr-13 September 2, 2019 10:04 Inactive
@codeclimate
Copy link

codeclimate bot commented Sep 2, 2019

Code Climate has analyzed commit ae7a9dc and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 87.5% (70% is the threshold).

This pull request will bring the total coverage in the repository to 95.5% (-0.5% change).

View more on Code Climate.

@Alpha1202 Alpha1202 force-pushed the ft-rate-articles-166840975 branch from ae7a9dc to faf37c6 Compare September 3, 2019 08:08
@Cavdy Cavdy temporarily deployed to ah-nyati-frontend-pr-13 September 3, 2019 08:08 Inactive
add a temporary login form
add a temporary get all articles page
add read one article page
add a dropdown button
add the star rating  modal
add the an on-success notification
[Starts #166840975]
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.

5 participants