fix(wallet): Transactions Partner Modal Bugs #27071
Merged
+146
−170
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
Transactions Partner
consent modal would keep popping up when navigating to other tabs.Transactions Partner
consent modal would not pop up when navigating fromSwap, Send, Bridge or Deposit
Transactions Partner
consent modal would not pop up if you manually navigated to brave://wallet/crypto/fund-walletResolves brave/brave-browser#42984
Resolves brave/brave-browser#42785
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Test Modal Keeps Appearing Bug
Buy
screen, once theTransactions Partner
consent modal pops up click thex
button to close.Activity
.Buy
tab.Before:
Screen.Recording.36.mov
After:
Screen.Recording.35.mov
Test Modal Not Appearing when Manually navigating Bug
Transactions Partner
consent modal should appear.Before:
Screen.Recording.37.mov
After:
Screen.Recording.33.mov
Test Modal Not Appearing when navigating from Send/Swap/Bridge/Deposit Bug
Send, Swap, Bridge or Deposit
.Buy
tab.Transactions Partner
consent modal should appear.Before:
Screen.Recording.38.mov
After:
Screen.Recording.34.mov