History

fix(web): simplify the desktop-managed server update banner copy (#6549)

Theo Browne committed 3 months ago

Commit 59be6f71 parent 5304f3e2 files · +2 −2Browse files

Unified · Split

2 files
  1. apps/web/src/versionSkew.test.ts +1 −1
  2. apps/web/src/versionSkew.ts +1 −1
apps/web/src/versionSkew.test.ts+1−1
@@ -101,7 +101,7 @@ describe("versionSkew", () => { · 100 unmodified lines
101101 "Update the Remote server so they stay in sync.",
102102 );
103103 expect(serverUpdateGuidance("desktop-managed", "Desktop server")).toBe(
104− "The Desktop server is run by the T3 Code desktop app on its machine — update the desktop app there to sync them.",
104+ "Update the desktop app that runs the Desktop server.",
105105 );
106106 expect(serverUpdateGuidance(null, "Local server")).toBe(
107107 "Relaunch the Local server with the copied command to sync them.",
apps/web/src/versionSkew.ts+1−1
@@ -73,7 +73,7 @@ export function serverUpdateGuidance( · 72 unmodified lines
7373 case "respawn":
7474 return `Update the ${serverLabel} so they stay in sync.`;
7575 case "desktop-managed":
76− return `The ${serverLabel} is run by the T3 Code desktop app on its machine — update the desktop app there to sync them.`;
76+ return `Update the desktop app that runs the ${serverLabel}.`;
7777 default:
7878 return `Relaunch the ${serverLabel} with the copied command to sync them.`;
7979 }

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t