Skip to content

[Fix] Discord setup shows installation controls when user links account - #1023

Merged
brunobergher merged 1 commit into
developfrom
fix/discord-link-step-3j5hsz9s9v88y
Aug 3, 2026
Merged

[Fix] Discord setup shows installation controls when user links account#1023
brunobergher merged 1 commit into
developfrom
fix/discord-link-step-3j5hsz9s9v88y

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

  • Replaced installation diagnostics in the configured Discord onboarding state with a focused personal account-linking step.
  • Added the step title "Link Discord Account" while preserving the existing Continue gate until the user's Discord identity is linked.
  • Added focused coverage for users who arrive with Discord already configured and for the transition immediately after saving credentials.

Why this change was made

Users prompted to link their personal Discord account were shown workspace-level installation diagnostics and admin actions, obscuring the task they needed to complete.

Impact

Configured Discord onboarding now matches the focused account-linking experience used by other entry points, while Discord installation diagnostics remain available in Settings. Browser proof could not reach the configured state without valid local Discord bot credentials; the configured and post-save states are covered by the focused component tests.

@roomote-community

roomote-community Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed adb6386

@brunobergher
brunobergher marked this pull request as ready for review August 3, 2026 12:49
@brunobergher
brunobergher merged commit 99911da into develop Aug 3, 2026
18 checks passed
@brunobergher
brunobergher deleted the fix/discord-link-step-3j5hsz9s9v88y branch August 3, 2026 12:49
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.

2 participants