Repository navigation
Conversation
okapitula
marked this pull request as ready for review
October 21, 2026 10:45
okapitula
requested review from
jakubfilinger-b,
klaudia-blazyczek-blurify,
marek-chmielowski-blurify,
mp-blurify and
zaxovaiko
as code owners
October 21, 2026 10:45
zaxovaiko
removed request for
jakubfilinger-b,
klaudia-blazyczek-blurify,
mp-blurify and
zaxovaiko
October 21, 2026 10:45
okapitula
added this pull request to stack #339
October 21, 2026 10:47
okapitula
force-pushed
the
feat/sportsbook-restriction-read
branch
from
October 21, 2026 12:23
3d7496d to
757eb32
Compare
okapitula
force-pushed
the
feat/sportsbook-restriction-read
branch
from
October 21, 2026 14:02
757eb32 to
21609e3
Compare
okapitula
force-pushed
the
feat/sportsbook-restriction-read
branch
from
October 21, 2026 18:48
21609e3 to
0985f47
Compare
Collaborator
|
LGTM. I'm giving approve, but it would be good if other dev also reviews this PR |
klaudia-blazyczek-blurify
previously approved these changes
Oct 22, 2026
okapitula
force-pushed
the
feat/sportsbook-restriction-read
branch
from
October 22, 2026 15:44
0985f47 to
5b1a66f
Compare
okapitula
force-pushed
the
feat/sportsbook-restriction-read
branch
from
October 25, 2026 12:18
5b1a66f to
135adef
Compare
Admin UIs had no way to show whether a player is restricted until an admin changed it. The new GET route returns the current state under the sportsbook view grant, unrestricted when no restriction row exists.
okapitula
force-pushed
the
feat/sportsbook-restriction-read
branch
from
October 26, 2026 09:11
135adef to
3a1979c
Compare
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
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.
Stacked on #336. Review that PR first; this one retargets to
devonce it merges.Summary
Adds
GET /backoffice/sportsbook/players/{userId}/restriction, which returns a player's current sportsbook restriction. It sits under thesportsbook:viewgrant and returns an unrestricted state when no restriction is in place.Why
Admins could restrict a player or lift a restriction, but nothing showed the current state. A back office had to change the restriction before it could show whether the player was blocked. The new read uses the same
SportsbookRestrictionshape the restrict and lift routes return. Staff who can view the sportsbook can see the state, and changing it still needssportsbook:suspend. An unknown user id returns 404, the same as the write routes.Alternatives considered
sportsbook:suspendfor the read. Support and review staff need to see why a player can't bet without being able to change it, so the read uses the view grant.Risks
None. This adds a read-only route and changes no existing contract, table or migration. The player-existence check moved into a shared helper and now uses the strict identity lookup, so a database error during the check returns 500 instead of a false 404.