Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Live leaderboard #528

Open
wants to merge 7 commits into
base: staging
Choose a base branch
from
Open

Conversation

Nydauron
Copy link
Contributor

@Nydauron Nydauron commented Feb 2, 2023

This is mainly for the Siebel dashboard, but this is a new public endpoint: WS /profile/live/leaderboard/

Once the connection has been established, the API will send back an initial leaderboard. From then on, the API will send back updated leaderboards whenever it detects updates to player point totals (primarily from POST /profile/points/award/). The amount of leaderboards the API can send back due to point value updates is 1 response/second.

The API will also send back an updated leaderboard every 5 minutes of inactivity.

  • Basic connection
  • Connect then send POST /profile/points/award/ POST /event/checkin/
  • Connect then update limit

Some other tests to consider t add in the future:

  • Connect then update limit then send POST /profile/points/award/ (or POST /event/checkin/)
  • Connect 10 clients and send POST /profile/points/award/ (or POST /event/checkin/)

@Nydauron Nydauron marked this pull request as ready for review February 22, 2023 08:36
@Timothy-Gonzalez
Copy link
Member

Haven't looked over the code yet - but do we really need to be within 1 second live? We could probably get away with once every 10 seconds or every 5 minutes, whichever is more often. It wouldn't really matter other than just not using resources we don't need too.

@asehgal4 asehgal4 self-requested a review February 23, 2023 02:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants