Skip to content

fix(skins): better offline handling#6344

Open
IMB11 wants to merge 1 commit into
mainfrom
cal/skins-offline-handling
Open

fix(skins): better offline handling#6344
IMB11 wants to merge 1 commit into
mainfrom
cal/skins-offline-handling

Conversation

@IMB11

@IMB11 IMB11 commented Jun 9, 2026

Copy link
Copy Markdown
Member
  • Makes the skins page read only and falls back to the default skin if it can't determine what the last skin applied was/api down.

@IMB11 IMB11 requested a review from a team June 9, 2026 11:03
@modrinth-bot

modrinth-bot commented Jun 9, 2026

Copy link
Copy Markdown
Member

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

  • Fixed issue with the Skin selector appearing in a broken state when the Minecraft api is unavailable. Now it appears in a read only state, showing the last selected skin or the default skin if it is unable to determine the last selected skin.

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@modrinth-bot

Copy link
Copy Markdown
Member

@IMB11 IMB11 added app Relates to Modrinth App frontend Involves work from the frontend team labels Jun 9, 2026

@tdgao tdgao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Development

Successfully merging this pull request may close these issues.

3 participants