-
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
#161966623: add reading time estimation #39
Conversation
f405dd8
to
7d6cf99
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.
The reading time is okey
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.
Good and simple implementation. It looks okay.
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.
Looks good. Please solve the current conflicts.
- estimate reading time for an article [Delivers #161966618]
7d6cf99
to
96c1269
Compare
Code Climate has analyzed commit 96c1269 and detected 0 issues on this pull request. View more on Code Climate. |
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 pull request do?
This PR estimates the reading time of an article.
Description of the tasks to be completed?
How should this be manually tested?
testing:
$ ./manage.py test
Screenshots:
Show the time taken to read an article
![image](https://user-images.githubusercontent.com/29709981/50282948-6ee1a380-0465-11e9-8bd6-b8cea6041939.png)
What are the relevant Pivotal Tracker Stories
#161966623
Checklist: