Skip to content

Conversation

@stadust
Copy link
Owner

@stadust stadust commented May 30, 2025

With player claims being mostly in place, and google oauth at least saving us from "I forgot my password", we can start implementing some much requested features traditionally home to the demonlist+ extension into pointercrate itself. For starters, this PR implements the "progress on overview" stuff, based on player claims (the claim does not need to be verified for this).

License Acceptance

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

stadust added 10 commits May 30, 2025 18:32
When logged in, make the player claimed by the logged in user available
to the demonlist overview page. Explicitly do not require a verified
claim.

Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
This way it's easier to access the claimed player.

Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
- Stop it being italic
- Add a "published by" text
- make name a statsviewer link

Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
Add the minimal possible score (for meeting list%) and total possible
score to the overview page.

Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
Similar to what demonlist+ is doing, display the progress of the player
claimed by the logged in user on the right hand side.

Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
It just doesnt fit width-wise

Signed-off-by: stadust <43299462+stadust@users.noreply.github.com>
@stadust stadust enabled auto-merge (squash) May 30, 2025 18:27
@stadust stadust merged commit 4e68cc1 into master May 30, 2025
2 checks passed
@codecov
Copy link

codecov bot commented May 30, 2025

Codecov Report

Attention: Patch coverage is 0% with 33 lines in your changes missing coverage. Please review.

Project coverage is 32.07%. Comparing base (d0928ce) to head (b7f0fc8).
Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
pointercrate-demonlist-pages/src/overview.rs 0.00% 20 Missing ⚠️
pointercrate-demonlist-api/src/pages.rs 0.00% 9 Missing ⚠️
pointercrate-demonlist/src/demon/mod.rs 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #260      +/-   ##
==========================================
- Coverage   32.16%   32.07%   -0.10%     
==========================================
  Files         118      118              
  Lines        8133     8157      +24     
==========================================
  Hits         2616     2616              
- Misses       5517     5541      +24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants