Fix/ranking issue - #67
Merged
Merged
Conversation
thinkDifferent1997
commented
Aug 11, 2026
thinkDifferent1997
commented
Owner
- translation to english
- profile pictures from api rendered in ranking and chat and page profile
The F5 guard read the navigation type at GameRoute mount, but that entry stays "reload" for the document's whole lifetime, so the first click into any game mode after any refresh got bounced home. Decide once at bundle load, against the URL actually reloaded, before BrowserRouter reads it. Also unblock the build: PartyGame had duplicate setIsInGame/joinSentRef declarations from the #56/#57 merge, and the prod compose was missing the SELinux ,z relabel on the cert mounts that dev already had.
…eaderboard bug with profile pic
Avatars were rendered as plain text, which worked for emoji avatars from email/password signup but printed the raw URL for OAuth users, breaking the layout on the leaderboard, chat and profile pages. - add reusable Avatar component that detects URLs vs emoji - use object-cover + overflow-hidden so remote images stay in the circle - set referrerPolicy=no-referrer to avoid 403s on Google-hosted avatars - fall back to the default emoji when an image fails to load
…ile page JwtStrategy already fetched the full user record but omitted from the object attached to req.user, so /api/auth/me never returned it. The navbar and profile page both fell back to the default emoji even for accounts with a valid OAuth picture. - add to AuthenticatedRequestUser and the strategy's return - fetch it in ProfilePage on mount for the current user
thinkDifferent1997
requested review from
EstebanGondim,
Manewa and
jacobscuzzi
August 11, 2026 18:21
Manewa
approved these changes
Aug 11, 2026
Manewa
left a comment
Manewa
left a comment
Collaborator
There was a problem hiding this comment.
ca merge ouuuuuuuuuuuuuuuuuuuuuuuuu
EstebanGondim
approved these changes
Aug 11, 2026
EstebanGondim
left a comment
EstebanGondim
left a comment
Collaborator
There was a problem hiding this comment.
Pull Request SUCCESS !
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.