Skip to content

feat(ui): Restrict shot upload according to title id and community#403

Merged
ashquarky merged 12 commits intodevfrom
work/upload-controls
Mar 9, 2026
Merged

feat(ui): Restrict shot upload according to title id and community#403
ashquarky merged 12 commits intodevfrom
work/upload-controls

Conversation

@ashquarky
Copy link
Member

Resolves #XXX

Changes:

  • Only allow posting screenshots in their designated communities. This is checked based on the community's title_id and the suspended software's title ID (from the param pack).
  • Extra title IDs can be provided in a new shot_extra_title_ids field. This is for games that don't have communities yet but we want to allow screenshots in a related community; without polluting the API response for title IDs.
  • A new shot mode toggle is added. This allows entirely blocking screenshots in a community (e.g. 3DS games that use the cameras) or forcing screenshots to be enabled (e.g. MK7 or PPT that disable screenshots for no reason).

This is ready for review but DO NOT merge until #378 has been in prod for a week or so - mods will need time to audit and fix the title IDs on our existing communities (plenty of them are wrong).

This is the final major feature needed for screenshot upload on CTR to be submitted.

@ashquarky ashquarky requested a review from mrjvs March 5, 2026 08:22
@ashquarky ashquarky mentioned this pull request Mar 6, 2026
4 tasks
@ashquarky ashquarky requested a review from mrjvs March 9, 2026 10:52
Copy link
Contributor

@mrjvs mrjvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment hasnt been resolved yet. For the rest it looks good!

(also soz, you have merge conflicts)

@ashquarky ashquarky requested a review from mrjvs March 9, 2026 12:45
Copy link
Contributor

@mrjvs mrjvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ashquarky ashquarky merged commit 038ffe8 into dev Mar 9, 2026
11 checks passed
@ashquarky ashquarky deleted the work/upload-controls branch March 9, 2026 13:08
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