Skip to content

fix: skins QA problems + flow change#6216

Open
IMB11 wants to merge 19 commits into
mainfrom
cal/skins-out-of-beta
Open

fix: skins QA problems + flow change#6216
IMB11 wants to merge 19 commits into
mainfrom
cal/skins-out-of-beta

Conversation

@IMB11
Copy link
Copy Markdown
Member

@IMB11 IMB11 commented May 26, 2026

  • Mojang is now treated as source of truth while the local DB is just saved skins/default cape state.
  • Before replacing a Mojang-only skin, we save it locally so switching skins does not lose it.
  • Saved skin PK is now texture_key + variant with cape_id becoming only metadata, which fixes duplicate skin rows.
  • Other bug fixes + QOL - adding cache types to profile fetching
  • Removed "default cape" concept
  • Align with figma designs

@IMB11 IMB11 added app Relates to Modrinth App backend Involves work from the backend team frontend Involves work from the frontend team labels May 26, 2026
@modrinth-bot
Copy link
Copy Markdown
Member

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@IMB11 IMB11 requested review from a team May 27, 2026 13:10
@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented May 27, 2026

@IMB11 IMB11 marked this pull request as ready for review May 27, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app Relates to Modrinth App backend Involves work from the backend team frontend Involves work from the frontend team

Development

Successfully merging this pull request may close these issues.

2 participants