Skip to content

Conversation

@IMB11
Copy link
Member

@IMB11 IMB11 commented Jan 23, 2026

  • Prefetches data explicitly on the server side (edge network benefits), pre-populates the query cache for tanstack.
  • Only calls versions, dependencies and org endpoints if needed (and lazy loaded when they are needed, except org endpoint)
  • Fixes SSR issues with the members panel and NavTabs components.
  • Migrates all subpages to use DI.

Testing Checklist

  • /
  • /changelog
  • /versions
  • Download modal
  • Sidebar elements correct
  • /gallery
  • Follow btn works
  • Bookmark btn works
  • /settings - all work
    • Icon upload
    • Name change
    • summary change
    • visibility change
    • Delete project works
  • /settings/tags - all work
    • Add a tag
    • Remove a tag
    • Set a featured tag
    • remove a featured tag
  • /settings/description
  • /settings/version still works?
  • /settings/license
  • /settings/gallery
  • /settings/links
  • /settings/members
  • /settings/analytics
  • /settings/environment (for moderators)

@IMB11 IMB11 requested a review from a team January 23, 2026 12:29
@IMB11 IMB11 added website Relates to Modrinth.com web frontend frontend Involves work from the frontend team labels Jan 23, 2026
@IMB11 IMB11 changed the title feat: ssr fixes feat: ssr fixes + switch project page to tanstack Jan 23, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 23, 2026

@IMB11 IMB11 marked this pull request as ready for review January 23, 2026 19:58
@Prospector Prospector enabled auto-merge January 23, 2026 20:10
@Prospector Prospector disabled auto-merge January 23, 2026 20:11
@Prospector Prospector merged commit 986a7e6 into main Jan 23, 2026
6 checks passed
@IMB11 IMB11 deleted the cal/tanstack-project-page-optimisations branch January 24, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Involves work from the frontend team website Relates to Modrinth.com web frontend

Development

Successfully merging this pull request may close these issues.

4 participants