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

Save read time on article creation and update #46

Merged
merged 1 commit into from
Jun 21, 2019
Merged

Conversation

eliemugenzi
Copy link
Contributor

what does this PR do?

  • Save read time on Article creation and update

Description of a task to be completed

  • User should be able to create an article, and the system calculates its read time and save it to the database

How should this be manually tested?

  • Clone this branch, cd into it
  • Install dependencies using yarn install
  • Run the new migrations with yarn migrate
  • Run the project using yarn dev
  • Hit the endpoint /api/articles , and use POST request to create a new article.
  • Hit /api/articles/{slug} and use PUT request to update an article.
  • The two above steps save the article's read time.
  • If you hit /api/articles/{slug} using GET request, the response comes with article's read time.

});
console.log(readTime(body));
Copy link

Choose a reason for hiding this comment

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

Unexpected console statement no-console

@eliemugenzi eliemugenzi force-pushed the ch-readtime-fix branch 4 times, most recently from b0e5a8c to de97dda Compare June 21, 2019 08:42
@eliemugenzi eliemugenzi added Needs Merge Needs To merge and removed Needs Review labels Jun 21, 2019
@eliemugenzi eliemugenzi force-pushed the ch-readtime-fix branch 2 times, most recently from dac4c68 to ad525fa Compare June 21, 2019 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Merge Needs To merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants