-
Notifications
You must be signed in to change notification settings - Fork 8
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
#168373102 Fix rating stats and readtime estimation #95
Conversation
66d6ede
to
2bef8a4
Compare
2bef8a4
to
55c0f3d
Compare
@@ -27,6 +29,26 @@ export default (data) => { | |||
percentage: percentages[data.indexOf(element)] | |||
})); | |||
|
|||
if (data.length < MAX_DATA_LENGTH) { |
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.
@nkpremices a comment to describe what you're doing here would be helpful.
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.
Lemme do that @MemunaHaruna
@@ -1,4 +1,4 @@ | |||
import chai from 'chai'; | |||
import chai, { expect } from 'chai'; |
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.
@nkpremices can you add a test for the read time estimation fix?
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.
I don't know how I can add a test for that as I'm just rounding up @MCFrank16
Give me an idea please
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.
But I'm going to try
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.
@nkpremices I left a few comments
55c0f3d
to
f4fa393
Compare
What does this PR do?
readtime
always rounded up