Skip to content

[backport cloud/1.34] remove useless/misleading toast in topup dialog#7380

Merged
christian-byrne merged 1 commit intocloud/1.34from
backport-remove-useless-toast-cloud-1.34
Dec 11, 2025
Merged

[backport cloud/1.34] remove useless/misleading toast in topup dialog#7380
christian-byrne merged 1 commit intocloud/1.34from
backport-remove-useless-toast-cloud-1.34

Conversation

@christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Dec 11, 2025

Summary

Backports the fix from main that removes the misleading "purchase successful" toast that appears immediately after clicking the buy button in the credit top-up dialog.

Details

  • Purchase actually happens on Stripe page, not immediately after clicking
  • Toast was confusing users into thinking purchase completed when it hadn't
  • Only error toast remains for actual failures
  • Fixed merge conflict in useCoreCommands.ts by keeping cloud/1.34's SubgraphNode import

Related

┆Issue is synchronized with this Notion page by Unito

Removes a toast that says "Purchase successful" when clicking the "Add
credits" button -- that button just opens Stripe checkout in another
tab, so the toast is misleading and could be wrong.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-7375-fix-remove-useless-misleading-toast-in-topup-dialog-2c66d73d36508124bb65feaf7cf26712)
by [Unito](https://www.unito.io)

---------

Co-authored-by: GitHub Action <action@github.com>
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Dec 11, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 11, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)
  • backport

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.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch backport-remove-useless-toast-cloud-1.34

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

@christian-byrne christian-byrne changed the title Backport: remove useless/misleading toast in topup dialog [backport cloud/1.34] remove useless/misleading toast in topup dialog Dec 11, 2025
@christian-byrne christian-byrne added the backport Backporting a PR onto a release candidate label Dec 11, 2025
@github-actions
Copy link

github-actions bot commented Dec 11, 2025

🎭 Playwright Test Results

Some tests failed

⏰ Completed at: 12/11/2025, 03:34:11 PM UTC

📈 Summary

  • Total Tests: 307
  • Passed: 105 ✅
  • Failed: 172 ❌
  • Flaky: 20 ⚠️
  • Skipped: 10 ⏭️

📊 Test Reports by Browser


🎉 Click on the links above to view detailed test results for each browser configuration.

@github-actions
Copy link

github-actions bot commented Dec 11, 2025

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 12/11/2025, 03:17:10 PM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@christian-byrne christian-byrne merged commit 350044d into cloud/1.34 Dec 11, 2025
21 of 32 checks passed
@christian-byrne christian-byrne deleted the backport-remove-useless-toast-cloud-1.34 branch December 11, 2025 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant