Skip to content

Add customizable CTA label to feature announcements#2400

Merged
gary149 merged 3 commits into
mainfrom
claude/artifact-announcement-update-vcngow
Jul 1, 2026
Merged

Add customizable CTA label to feature announcements#2400
gary149 merged 3 commits into
mainfrom
claude/artifact-announcement-update-vcngow

Conversation

@gary149

@gary149 gary149 commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds support for customizable call-to-action (CTA) labels in feature announcements, allowing announcements to override the default "Learn more" text with custom labels like "Go to the model".

Preview

The home-screen announcement toast (light + dark):

Announcement toast preview

The preview image lives at .github/media/announcement-gemma-cerebras.png (outside the app bundle). Feel free to drop that commit before merge — or replace this with a drag-and-drop upload.

Changes

  • Interface update: Added optional cta?: string field to FeatureAnnouncement interface
  • Processing logic: Updated getActiveAnnouncement() to extract and trim the cta field from announcement JSON, treating blank/whitespace-only values as undefined
  • UI rendering: Modified FeatureAnnouncementToast.svelte to display custom CTA label when provided, falling back to "Learn more" default
  • Documentation: Updated .env and example configurations to document the new cta field
  • Test coverage: Added two new test cases:
    • Verifies trimmed CTA labels are preserved when provided
    • Verifies CTA remains undefined when missing or blank
  • Configuration updates: Updated dev and prod Helm chart configurations with new example announcement featuring the Gemma-4-31B model

Implementation Details

The CTA label processing uses the existing asTrimmedString() utility to handle whitespace normalization, ensuring consistent behavior with other announcement fields. Blank strings are converted to undefined to maintain a clean data model.

https://claude.ai/code/session_01PHfp4To6qTDrjFF5Dace1d

Swap the home-screen feature announcement toast from "Introducing
Artifacts" to a Gemma-4-31B / Cerebras inference message with a
call-to-action linking to the model page.

Adds an optional `cta` field to feature announcements so the toast link
label can be customized (defaults to "Learn more"), used here to render
"Go to the model".

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PHfp4To6qTDrjFF5Dace1d

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9c2c537ece

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread chart/env/prod.yaml
claude added 2 commits July 1, 2026 20:11
Light + dark render of the Gemma-4-31B / Cerebras announcement toast,
referenced from the PR description. Lives under .github/media so it is
outside the app bundle; safe to drop before merge.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PHfp4To6qTDrjFF5Dace1d
The toast rendered announcement.link directly as the anchor href, so an
app-relative link like /models/... resolved without APP_BASE (e.g. /chat)
in prod, sending the CTA to the wrong origin. Prepend `base` from
$app/paths for internal links, matching how the rest of the app builds
model links; external URLs are left untouched.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PHfp4To6qTDrjFF5Dace1d
@gary149 gary149 merged commit b08e2ba into main Jul 1, 2026
6 checks passed
@gary149 gary149 deleted the claude/artifact-announcement-update-vcngow branch July 1, 2026 20:22
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