Skip to content

feat(discord/build): one code block per tag in copy reply#151

Merged
1 commit merged intomasterfrom
worktree-build-copy-tag-per-block
Apr 21, 2026
Merged

feat(discord/build): one code block per tag in copy reply#151
1 commit merged intomasterfrom
worktree-build-copy-tag-per-block

Conversation

@barnabasbusa
Copy link
Copy Markdown
Contributor

Summary

  • "Copy tag" button now returns an ephemeral reply with each image tag in its own triple-backtick block, so Discord's per-block copy icon grabs exactly one tag — no duplicated link+block, no risk of copying every tag at once.
  • Button pluralizes to "Copy tags" when a build produces more than one image.
  • "Show another image…" dropdown removed (redundant — all tags are already in the reply). dockerImage.Label() dropped with it.

Test plan

  • Trigger a single-image build, click Copy tag in the DM, verify the ephemeral reply is one code block and Discord's copy icon yields the tag.
  • Trigger a multi-image build (variants), verify button label reads Copy tags, reply contains one code block per image, and each code block copies its own tag independently.
  • Confirm no dropdown is rendered alongside the button.
  • Stale-cache path: restart the bot or wait past the completion TTL, click the button, verify the "no longer in cache" ephemeral still fires.

Clicking "Copy tag"/"Copy tags" now returns an ephemeral reply with
each image tag in its own triple-backtick block, so Discord's per-block
copy icon copies a single tag — no more duplicated link+block and no
risk of grabbing every tag at once. The "Show another image…" dropdown
is removed since all tags are visible in the reply.
@barnabasbusa barnabasbusa requested a review from mattevans as a code owner April 21, 2026 08:04
@barnabasbusa barnabasbusa closed this pull request by merging all changes into master in 43c7dad Apr 21, 2026
@barnabasbusa barnabasbusa deleted the worktree-build-copy-tag-per-block branch April 21, 2026 08: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.

1 participant