Skip to content

[Improve] Clarify setup integration history - #2539

Open
roomote-roomote[bot] wants to merge 1 commit into
developfrom
improve/setup-integration-history-2rv9qvia3cjxq
Open

[Improve] Clarify setup integration history#2539
roomote-roomote[bot] wants to merge 1 commit into
developfrom
improve/setup-integration-history-2rv9qvia3cjxq

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Created by Roomote. Follow up by mentioning @roomote-roomote, in the web UI, or in Slack.

What changed

  • Label completed setup integration discovery as Asked about integrations in conversation history.
  • Keep the existing plug treatment consistent between the dedicated integration card and its completed history item.
  • Add focused coverage for the card icon, persisted receipt metadata, and rendered history copy/icon.

Why this change was made

The setup integration flow already follows source control and precedes starter-task suggestions. This follow-up makes its completed state use the intended conversational language and visual identity.

Impact

Users see Asked about integrations after completing or skipping the optional integration step. The existing category order, recommendations, Connect actions, and first-class Keep going path are unchanged.

Validation

  • Server setup-session tests: 12 passed.
  • Client setup-card and transcript tests: 87 passed.
  • Web TypeScript check and targeted ESLint passed.
  • Pre-push oxlint, residual lint, fast typechecks, and knip passed.

Screenshots

The real setup component was rendered with representative integration options for layout validation; these screenshots do not prove backend discovery or OAuth behavior.

Setup integrations card on desktop

Setup integrations card on mobile

@roomote-community

roomote-community Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed e493184

@roomote-roomote
roomote-roomote Bot marked this pull request as ready for review September 11, 2026 18:12
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