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

Add average performance #102

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

PrenilS
Copy link

@PrenilS PrenilS commented Aug 6, 2021

First of all, I apologize in advance if this is the incorrect way to submit a pull request. Its my first time doing one on GitHub.

I thought that it would be interesting to be able to see your average accuracy and WPM so this branch implements that saving the sum of total accuracy and WPM scores as well as total tests completed inside cookies and displaying the resulting averages below the main text box.

Cookies are still set to a 90 day expiry so stats will reset after that time but of course, that can be extended if it makes sense to (I'm not clear on what the best practice for storing cookies over long periods of time is).

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.

1 participant