Skip to content

Add average performance#102

Open
PrenilS wants to merge 2 commits intobriano1905:masterfrom
PrenilS:Add-average-performance
Open

Add average performance#102
PrenilS wants to merge 2 commits intobriano1905:masterfrom
PrenilS:Add-average-performance

Conversation

@PrenilS
Copy link
Copy Markdown

@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