Skip to content

ci: use the dino as the bot avatar instead of a body image - #338

Merged
JonasJesus42 merged 1 commit into
mainfrom
ci-discord-image-as-avatar
Sep 2, 2026
Merged

ci: use the dino as the bot avatar instead of a body image#338
JonasJesus42 merged 1 commit into
mainfrom
ci-discord-image-as-avatar

Conversation

@JonasJesus42

@JonasJesus42 JonasJesus42 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #337. As an embed image the art rendered full-width under the commands, pushing the thing people actually came for out of view — on a message that fires on every merge to main.

Same asset, now the bot avatar_url: the art still shows, the body goes back to being just the version and the two update paths.

Keeping it in the payload rather than relying on the webhook's avatar configured in Discord means it survives a webhook being recreated, and the committed asset stays justified.

Verified: publish.yml parses (11 steps), payload has avatar_url and no embeds[].image.


Summary by cubic

Release notifications now show the dino as the Discord bot avatar instead of a full-width embed image, keeping the version and update links prominent. The workflow sends the avatar URL in each payload, so the branding does not depend on Discord webhook configuration.

Written for commit 87f6eb6. Summary will update on new commits.

Review in cubic

As an embed image it rendered full-width under the commands, pushing the thing
people actually came for out of view -- on a message that fires every merge.

Same asset as avatar_url: art still shows, body is just the version and the two
update paths. In the payload rather than Discord-side config so it survives the
webhook being recreated.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@JonasJesus42
JonasJesus42 merged commit 64f4027 into main Sep 2, 2026
2 checks passed
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