refactor: migrate buy button toast#44510
Open
n3ps wants to merge 1 commit into
Open
Conversation
Contributor
Builds ready [f748e32]
⚡ Performance Benchmarks (Total: 🟢 16 pass · 🟡 8 warn · 🔴 0 fail)
Bundle size diffs
|
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 19:11
f748e32 to
45d350a
Compare
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 19:14
45d350a to
543ec53
Compare
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 19:21
543ec53 to
f748e32
Compare
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 19:43
f748e32 to
e12f2d6
Compare
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 19:48
e12f2d6 to
6657069
Compare
Contributor
Builds ready [6657069]
⚡ Performance Benchmarks (Total: 🟢 12 pass · 🟡 10 warn · 🔴 2 fail)
Bundle size diffs
|
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 22:59
6657069 to
cc969c6
Compare
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 23:04
cc969c6 to
43963f1
Compare
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 17, 2026 23:07
43963f1 to
c927266
Compare
Contributor
Builds ready [c927266]
⚡ Performance Benchmarks (Total: 🟢 16 pass · 🟡 8 warn · 🔴 0 fail)
Bundle size diffs
|
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 18, 2026 04:35
c927266 to
aa3b6ee
Compare
n3ps
changed the title
n3ps
marked this pull request as ready for review
July 18, 2026 04:39
n3ps
enabled auto-merge
July 18, 2026 04:39
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit aa3b6ee. Configure here.
| <Icon name={IconName.Export} color={IconColor.IconDefault} /> | ||
| ), | ||
| }, | ||
| ); |
There was a problem hiding this comment.
Stale locale in buy toast
Low Severity
Buy-tab toast strings are resolved via t() inside handleBuyAndSellOnClick, but that callback’s dependency list omits t. After the user changes locale, a Buy click can show the previous language because the memoized handler still closes over the old t, unlike the removed toast component that read t on render.
Reviewed by Cursor Bugbot for commit aa3b6ee. Configure here.
Co-authored-by: Francis Nepomuceno <n3ps@users.noreply.github.com>
cursor
Bot
force-pushed
the
n3ps/ceux-1188-50fc
branch
from
July 18, 2026 04:44
aa3b6ee to
e3b5cf0
Compare
|
Contributor
Builds ready [e3b5cf0]
⚡ Performance Benchmarks (Total: 🟢 11 pass · 🟡 13 warn · 🔴 0 fail)
Bundle size diffs
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Migrates the toast notification when clicking the homescreen Buy button
Changelog
CHANGELOG entry: null
Related issues
Fixes: CEUX-1188
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
UI-only notification refactor with unchanged trigger conditions (non-ramps buy path); no auth, data, or payment logic changes.
Overview
Replaces the wallet overview Buy “tab opened” notification with the shared
toast.successstack instead of a local multichainToast+ component state.When ramps are disabled and Buy opens Portfolio in a new tab, the same copy and export icon are shown via
ToastContent, with a stable toast id (buy-tab-opened-toast). The inlineTabOpenedToasthelper andshowTabOpenedToaststate/render path are removed.Reviewed by Cursor Bugbot for commit e3b5cf0. Bugbot is set up for automated code reviews on this repo. Configure here.