Skip to content

Add successVariant prop to ConfirmDialog for customizable toast types#1113

Merged
gciotola merged 1 commit into
mainfrom
fix/1112-confirm-dialog-toast-variant
May 7, 2026
Merged

Add successVariant prop to ConfirmDialog for customizable toast types#1113
gciotola merged 1 commit into
mainfrom
fix/1112-confirm-dialog-toast-variant

Conversation

@gciotola
Copy link
Copy Markdown
Contributor

@gciotola gciotola commented May 7, 2026

Closes #1112

What I did

Fulfilled toast has default variant, instead of success, but it can also be overridden when needed.

How to test

Checklist

  • Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests).
  • Make sure to add/update documentation regarding your changes.
  • You are NOT deprecating/removing a feature.

…types

Co-authored-by: Copilot <copilot@github.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 7, 2026

Deploy Preview for commercelayer-app-elements ready!

Name Link
🔨 Latest commit 979c29d
🔍 Latest deploy log https://app.netlify.com/projects/commercelayer-app-elements/deploys/69fc78923e4e000008993e47
😎 Deploy Preview https://deploy-preview-1113--commercelayer-app-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@gciotola gciotola changed the title fix: add successVariant prop to ConfirmDialog for customizable toast … Add successVariant prop to ConfirmDialog for customizable toast types May 7, 2026
@gciotola gciotola self-assigned this May 7, 2026
@gciotola gciotola added the bug Something isn't working label May 7, 2026
@gciotola gciotola merged commit d6cd4a5 into main May 7, 2026
7 checks passed
@gciotola gciotola deleted the fix/1112-confirm-dialog-toast-variant branch May 7, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve useConfirmDialog

2 participants