Skip to content

Clear stale provider update toast CTA#2755

Draft
jeremy0dell wants to merge 1 commit into
pingdotgg:mainfrom
jeremy0dell:feature/provider-update-toast
Draft

Clear stale provider update toast CTA#2755
jeremy0dell wants to merge 1 commit into
pingdotgg:mainfrom
jeremy0dell:feature/provider-update-toast

Conversation

@jeremy0dell
Copy link
Copy Markdown

@jeremy0dell jeremy0dell commented May 19, 2026

Summary

  • Clear stale toast action props when provider update progress toasts move into loading or success states.
  • Keep failed provider update toasts actionable via Settings.
  • Add regression coverage for Base UI shallow-merge behavior.

Root Cause

Base UI shallow-merges toast updates. The initial provider update prompt sets the primary CTA to Update, and the running toast update previously omitted actionProps, so the old CTA remained visible on the Updating providers toast.

Validation

  • bun run test
  • bun fmt
  • bun lint
  • bun typecheck

Note

Clear stale action buttons from provider update toast on progress and success

  • Adds buildProviderUpdateToastUpdate in ProviderUpdateLaunchNotification.logic.ts to centralize toast update construction for provider update views.
  • For loading/success views, sets actionProps: undefined explicitly to clear any action buttons (e.g. 'Update') left over from a previously shown prompt toast, working around shallow merge behavior in the toast manager.
  • For other views (e.g. rejected), attaches a 'Settings' action button with outline styling.
  • Refactors updateProviderUpdateToast in ProviderUpdateLaunchNotification.tsx to call the new helper instead of duplicating inline logic.

Macroscope summarized edad6db.

- Factor toast update construction into shared logic
- Remove prompt actions from loading and success update states
- Keep rejected update states actionable via Settings
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 64f6490a-5e7f-498d-a29a-2942283ddfd3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant