-
Notifications
You must be signed in to change notification settings - Fork 4
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
#159987724 Users should be able to see time it takes to read an article #42
Conversation
); | ||
}, | ||
|
||
down: (queryInterface) => { |
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.
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>
arrow-body-style
@@ -0,0 +1,18 @@ | |||
module.exports = { | |||
up: (queryInterface, Sequelize) => { |
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.
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>
arrow-body-style
Extended the article model to include timeToRead column. Calculate time to read article then save article. - Add calculateTimeToRead helper method - Add pre save hook - include relevant tests
c419885
to
c841dad
Compare
Pull Request Test Coverage Report for Build 283
💛 - Coveralls |
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?
add read time to an article.
Description of Task to be completed?
Extended the article model to include
timeToRead
column. Calculate timeto read an article then save the article.
How should this be manually tested?
Run
npm run migrate
Run
npm run start:dev
send a
POST
request to/api/articles/
to create an articleWhat are the relevant pivotal tracker stories?
#159987724