-
Notifications
You must be signed in to change notification settings - Fork 857
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(Wallet) Clean up and improve transaction fragment implementation #25628
base: master
Are you sure you want to change the base?
Conversation
6e78e86
to
ec8538d
Compare
[puLL-Merge] - brave/brave-core@25628 DescriptionThis PR makes significant changes to the Brave Wallet functionality in the Android app. It primarily focuses on improving the UI/UX of transaction approval and details views, refactoring code for better maintainability, and addressing potential issues. ChangesChanges
Possible Issues
Security Hotspots
|
Resolves brave/brave-browser#41122
Demo
screen-20240918-142544.mp4
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: