Skip to content

feat: servers in app finalization#5744

Merged
tdgao merged 154 commits intotruman/new-server-settingsfrom
cal/dev-884
Apr 12, 2026
Merged

feat: servers in app finalization#5744
tdgao merged 154 commits intotruman/new-server-settingsfrom
cal/dev-884

Conversation

@IMB11
Copy link
Copy Markdown
Member

@IMB11 IMB11 commented Apr 3, 2026

No description provided.

IMB11 and others added 10 commits March 30, 2026 14:27
* feat: implement shared server header for app and website

* feat: implement wrapped overview page with shared composable and hook it up

* pnpm prepr

* fix: bugs

* qa: cleanup

* feat: root.vue shared layout

* feat: delete old options pages + fix discovery frontend

* fix: discovery

* fix: misc style/layout issues

* fix page padding

* fix: modal height jankiness

* feat: implement server install content in app and server setup modal with DI

* fix: spacing

* remove servers in app feature flag

* Revert "remove servers in app feature flag"

This reverts commit 86e284c.

* fix: qa

* feat: remove legacy components from apps/frontend/src/components/ui/servers

---------

Co-authored-by: Calum H. (IMB11) <contact@cal.engineer>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 3, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
storybook e578934 Commit Preview URL

Branch Preview URL
Apr 12 2026, 12:34 AM

@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

@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented Apr 3, 2026

@IMB11 IMB11 marked this pull request as ready for review April 11, 2026 23:22
Copy link
Copy Markdown
Contributor

@tdgao tdgao left a comment

Choose a reason for hiding this comment

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

LGTM

@tdgao tdgao merged commit b36647f into truman/new-server-settings Apr 12, 2026
6 of 7 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