Skip to content

[Fix] Discord review resolutions appear as full-size text - #1305

Merged
daniel-lxs merged 2 commits into
developfrom
fix/discord-review-resolution-subtext-3mw0jjooyc72m
Aug 13, 2026
Merged

[Fix] Discord review resolutions appear as full-size text#1305
daniel-lxs merged 2 commits into
developfrom
fix/discord-review-resolution-subtext-3mw0jjooyc72m

Conversation

@roomote-community

@roomote-community roomote-community Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

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

What changed

Discord PR review action resolutions now use Discord's -# subtext syntax when replacing the action buttons, including notifications whose original content is empty. Message truncation accounts for the formatting prefix, and focused coverage verifies the auto-resolve copy, empty-content behavior, and maximum message length.

Why this change was made

Discord displayed the action resolution as full-size body text, while Slack presents the equivalent status as compact context. This makes the two integration experiences visually consistent.

Impact

PR review action confirmations in Discord render as smaller, quieter status text beneath the preserved review notification, or by themselves when there is no notification body, without exceeding Discord's message limit.

@roomote-community

roomote-community Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

No issues outstanding. See task

  • apps/api/src/handlers/discord/pr-review-action.ts:47 Empty Discord notification content bypasses the new subtext prefix.

Reviewed 400e53e

Comment thread apps/api/src/handlers/discord/pr-review-action.ts Outdated
@daniel-lxs
daniel-lxs marked this pull request as ready for review August 13, 2026 23:59
@daniel-lxs
daniel-lxs merged commit 5c8eb84 into develop Aug 13, 2026
18 checks passed
@daniel-lxs
daniel-lxs deleted the fix/discord-review-resolution-subtext-3mw0jjooyc72m branch August 13, 2026 23:59
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