Skip to content

Commit 8770459

Browse files
committed
Point the per-action hint and the memory sync messages at Connections
Main's Arcade work and its memory sources still sent people to Settings, Connected accounts, which now only redirects to Connections.
1 parent 94a62b0 commit 8770459

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

‎app/src/lib/plugins/provider-vocabulary.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ export function brokeredConsentTrip(provider: PluginsProvider): string {
171171
* that is not there.
172172
*
173173
* `where` IS NOT DECORATION AND HAS NO DEFAULT. The two screens that draw this row do not both
174-
* draw the action list: the settings screen has it directly underneath, and the admin setup page
174+
* draw the action list: the Connections page has it directly underneath, and the admin setup page
175175
* has none at all, so a single sentence saying "below" is a true direction on one screen and a
176176
* wrong one on the other — pointing somebody at a list that is not on their page is the same
177177
* failure as the Connect button this sentence replaces, one step further on. Each caller says
@@ -184,7 +184,7 @@ export function brokeredPerActionTrip(
184184
const stem = `${VOCABULARY[provider].name} approves one action at a time, so there is nothing to connect for the whole app.`;
185185
return where === "below"
186186
? `${stem} Authorize the actions you want below.`
187-
: `${stem} Authorize them from Connected accounts in Settings.`;
187+
: `${stem} Authorize them from Connections.`;
188188
}
189189

190190
/** What a person is told where the deployment has no key for the active broker. */

‎server/src/memory/ingestion.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -306,9 +306,9 @@ export function createMemoryIngestion(options: {
306306
case "shared":
307307
return `The shared ${prompt.title} account is not connected. An administrator has to connect it before this source can sync.`;
308308
case "reconnect":
309-
return `Your ${prompt.title} account needs reconnecting. Reconnect it under Settings, Connected accounts, then sync again.`;
309+
return `Your ${prompt.title} account needs reconnecting. Reconnect it under Connections, then sync again.`;
310310
default:
311-
return `Your ${prompt.title} account is not connected. Connect it under Settings, Connected accounts, then sync again.`;
311+
return `Your ${prompt.title} account is not connected. Connect it under Connections, then sync again.`;
312312
}
313313
}
314314

‎server/tests/memory-store.integration.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ test("a refusal carrying an app card is said for the sources page, not stored ra
274274
);
275275
const { syncError } = await store.getSource(owner, value.id);
276276
expect(syncError).toBe(
277-
"Your Docs account needs reconnecting. Reconnect it under Settings, Connected accounts, then sync again.",
277+
"Your Docs account needs reconnecting. Reconnect it under Connections, then sync again.",
278278
);
279279
expect(syncError).not.toContain("[connect-app]");
280280
} finally {

0 commit comments

Comments
 (0)