Skip to content

qa: shared instances post release - #6893

Merged
IMB11 merged 16 commits into
mainfrom
cal/post-release-shared-instances
Jul 28, 2026
Merged

qa: shared instances post release#6893
IMB11 merged 16 commits into
mainfrom
cal/post-release-shared-instances

Conversation

@IMB11

@IMB11 IMB11 commented Jul 27, 2026

Copy link
Copy Markdown
Member

see individual commits + changelog

@modrinth-bot

modrinth-bot commented Jul 27, 2026

Copy link
Copy Markdown
Member

Note

This changelog has been baked. Any further edits will not be reflected.

Pull request changelog

App

Added

  • Added a banner for users of a shared instance which let's them know that they need to review an update to play the instance - alongside the existing checks when you click Play.
  • Added a way to see and manage the invite links you have created for a shared instance in the Sharing tab of the Instance Settings modal.
  • Added a way to see where a shared instance content is coming from when in the Install to play modal, either the linked modpack or if it was added on top. Content which is a part of the linked modpack will show the modpack information underneath it's name.

Changed

  • Limited shared instances to 50 users.
  • Changed the expiry date picker in the shared instance invite edit modal to be a dropdown of common dates, rather than a complicated date picker. You can still use the fine-grained date picker by choosing "Custom"
  • Instance icons must now be smaller than 4MB - any existing instances will have their icons compressed to 512x512px size to conform to the new limit. This will break any instances which have .GIF icons.

Deprecated

Removed

Fixed

  • Fixed error spam when the shared instances API is not accessible, it will cleanly provide feedback that the app cannot connect.

Security

Website

Added

Changed

Deprecated

Removed

Fixed

  • Fixed shared instance reports not showing up in the dashboard's Reports page.
  • Fixed shared instance report emails saying "Unknown" rather than the shared instance's name.

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@IMB11
IMB11 marked this pull request as draft July 27, 2026 11:13
@modrinth-bot

modrinth-bot commented Jul 27, 2026

Copy link
Copy Markdown
Member

@IMB11
IMB11 marked this pull request as ready for review July 27, 2026 17:09
@IMB11
IMB11 requested review from a team and Prospector and removed request for Prospector July 27, 2026 17:09
@IMB11
IMB11 enabled auto-merge July 28, 2026 12:24
IMB11 added 2 commits July 28, 2026 13:30
…ed-instances

# Conflicts:
#	apps/app-frontend/src/pages/instance/share/index.vue
#	apps/app-frontend/src/pages/instance/share/shared-instance-members-table.vue
@IMB11
IMB11 added this pull request to the merge queue Jul 28, 2026
Merged via the queue into main with commit 82348fe Jul 28, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants