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 Global Scoreboard #161

Closed
jsolly opened this issue May 15, 2023 · 3 comments · Fixed by #171
Closed

Add Global Scoreboard #161

jsolly opened this issue May 15, 2023 · 3 comments · Fixed by #171
Assignees
Labels
functional_requirement Something tangible the user will notice priority_high

Comments

@jsolly
Copy link
Owner

jsolly commented May 15, 2023

Context

Users should be able to see the global top 10 scores when the game ends

Current Behavior/Workaround

Only showing current user's top score via local storage

Things to consider

Store in a flat file or database?
Any issues with concurrency if using a flat file?
Will need to provide a way to add three-letter username to associate with score
Will need placeholders for the top 10 scores to start

@jsolly
Copy link
Owner Author

jsolly commented May 15, 2023

@shawnamays do you have any suggestions for what should go in the scoreboard or other enhancements related to it?

@jsolly jsolly added functional_requirement Something tangible the user will notice priority_high labels May 20, 2023
@jsolly jsolly self-assigned this May 20, 2023
@jsolly jsolly linked a pull request May 20, 2023 that will close this issue
2 tasks
@jsolly
Copy link
Owner Author

jsolly commented Jun 19, 2023

Implementation finished

  • Using a MongoDB database for the high scores
  • Serverless functions for GET/POST to api/highscores
  • The name for each highscore is a three letter capitalized name
  • The database only maintains the top 10 scores

@shawnamays
Copy link

shawnamays commented Jun 19, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
functional_requirement Something tangible the user will notice priority_high
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants