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

fix: use selected typing speed unit on personal best popup (fehmer) #5070

Merged
merged 13 commits into from
Feb 19, 2024

Conversation

fehmer
Copy link
Member

@fehmer fehmer commented Feb 16, 2024

Description

fixes #5059

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Feb 16, 2024
@fehmer fehmer force-pushed the feature/fix-pb-typing-unit branch 2 times, most recently from 3a44525 to 8d9bddb Compare February 16, 2024 10:48
@fehmer fehmer force-pushed the feature/fix-pb-typing-unit branch from 8d9bddb to dfd82fc Compare February 16, 2024 11:05
@fehmer fehmer marked this pull request as draft February 16, 2024 23:48
@fehmer fehmer changed the title fix: Use selected typing speed unit on personal best popup fix: Use selected typing speed unit on personal best popup (fehmer) Feb 19, 2024
@fehmer fehmer marked this pull request as ready for review February 19, 2024 09:58
@fehmer fehmer requested a review from Miodec February 19, 2024 10:04
@Miodec Miodec changed the title fix: Use selected typing speed unit on personal best popup (fehmer) fix: use selected typing speed unit on personal best popup (fehmer) Feb 19, 2024
@Miodec Miodec merged commit bac837d into master Feb 19, 2024
4 checks passed
@Miodec Miodec deleted the feature/fix-pb-typing-unit branch February 19, 2024 13:46
Miodec pushed a commit that referenced this pull request Feb 19, 2024
…5070)

* fix: Use selected typing speed unit on personal best popup

* refactor

* refactor

* test coverage

* use Format in more places

* Make config mockable

* dependency injection

* wip

* fix

* test

* touch
Miodec added a commit that referenced this pull request Feb 19, 2024
* move user to shared definitions

this includes whatever user can have on it, tags, presets and so on

* profiles, mail and others

* fix logic

* yeet

* same as master for now

* tsc fixes

* remove comment

* fix tests

* chore: omit ips

* fix(language): remove some unnecessarily capitalised words in german 1k

* fix(typing): first space sometimes soft locking the website

* perf: speed up settings page loading

* fix: use selected typing speed unit on personal best popup (fehmer) (#5070)

* fix: Use selected typing speed unit on personal best popup

* refactor

* refactor

* test coverage

* use Format in more places

* Make config mockable

* dependency injection

* wip

* fix

* test

* touch

* fix(language): typos in russian_10k.json (kae) (#5082)

* Update russian_10k.json

- fixed typos
- removed duplicates

* - fixed extra typos

* remove duplicates

* fix(language): typos in russian_10k.json

* feat: add copy missed words to result screen (fehmer) (#5086)

* feat: Add copy missed words to result screen

* remove margin

* update icons

---------

Co-authored-by: Miodec <jack@monkeytype.com>

* impr(funbox): add 46 group languages to wikipedia funbox (RealCyGuy) (#5078)

* impr: provide all-time LB results during LB update (fehmer) (#5074)

Try to provide LB results during the LB update. There is a very small time-frame where
already running queries might fail during the update. For now we keep the 503 error in this
cases and monitor how often this happens on production.

* impr(funbox): add ` (grave accent, 96) and ~ (tilde, 126) to specials (#5073)

* impr: add testWords and wordsHistory to copy result stats (#5085)

* feat: add testWords and wordsHistory to copy result stats

* fix

* add fe ts dep

---------

Co-authored-by: Christian Fehmer <fehmer@users.noreply.github.com>
Co-authored-by: Andrey Kuznetsov <akuznetsov@outlook.com>
Co-authored-by: Cyrus Yip <cyruscmyip1@gmail.com>
Co-authored-by: fitzsim <fitzsim@fitzsim.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend User interface or web stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use currently selected typing speed unit on the personal bests popup
3 participants