PR | Leaderboardlocalstorage

[dominiksakic]

Description

Refactor leaderboard into a CSR component to get access to the username. I used localstorage instead of the usecontext for ooptimization to not fetch. I stored the username only no other data in the localstorage.

When the user logs out the data gets destroyed. Also the user gets highlighted, @DepartureLV please still it to be better!

Card Link

https://3.basecamp.com/5802516/buckets/37608217/card_tables/cards/7498909855

Type of change

Please delete options that are not relevant.

• [x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• [x] Manually

Checklist before requesting a review

• [x] I have performed a self-review of my code
• [x] I have commented my code, particularly in hard-to-understand areas
• [x] I have added tests that prove my fix is effective or that my feature works
• [x] New and existing unit tests pass locally with my changes
• [x] Any dependent changes have been merged and published in downstream modules

[J-Ariola]

You can open this PR again when you finish fixing the issue.

[dominiksakic]

@Flumanuck

1. I got this error from the FRONTEND: The app didnt crash, and worked fine even though the error happened. Could we handle this error somehow?

2. I logged in but there is too many users on the leaderboard already and I cant see my name. Could you make the leaderboard scrollable?

[Flumanuck]

@dominiksakic I can take a look. I think I can make it scrollable. I'll look into the error too.

[dominiksakic]

Jacob is the leaderboard scrollable on your machine? On mine it isnt - I checked my local db and should have more users in the display