Skip to content

Make the Cloud Server step clear when a plan already covers the drive - #1912

Closed
joepio wants to merge 2 commits into
developfrom
claude/project-thread-u8vgm0-cloud-server-step
Closed

joepio wants to merge 2 commits into
developfrom
claude/project-thread-u8vgm0-cloud-server-step

Conversation

@joepio

@joepio joepio commented Sep 29, 2026

Copy link
Copy Markdown
Member

What

Before: on a drive with a plan or manual grant, the sync page still showed Cloud Server as off with the text "This drive already has a Server plan. Connecting it uses that plan...". After the dialog it said "Connected to Cloud Server. Syncing..." but the row stayed grey with a "Finish Cloud Server setup" button and no sign of progress.

After: with a plan the row says "Included in your plan" and the button is "Turn on Cloud Server" (it still opens the hosting consent dialog, since that is the legal step). Once enrolled but not yet hosted, the row says "Setting up Cloud Server..." and turns on by itself when the workspace has arrived; the button becomes "Sync again" instead of "Finish Cloud Server setup".

How

Wording and button labels in the off-state Cloud Server row of SyncRoute.tsx. The enrollment logic is unchanged. Lint passes; not verified against a live saas grant, and no screenshot yet.


Generated by Claude Code

joepio added a commit that referenced this pull request Oct 5, 2026
… chat, search, CI) (#2060)

Combines #1530, #1638, #1663, #1835, #1869, #1896, #1898, #1911, #1912, #1913, #1914, #1918, #1919, #1929, #1931, #1949, #1958, #1959, #1961, #1963, #1968, #1975, #1981, #1985, #1988, #2038, #2041, #2047 and #2050, tested in one full Mancave run on the exact head (run 4999, green).

joepio commented Oct 5, 2026

Copy link
Copy Markdown
Member Author

Landed on develop in the combined batch #2060 (95707c0), tested together in one full run. Closing as included.


Generated by Claude Code

@joepio joepio closed this Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant