-
Notifications
You must be signed in to change notification settings - Fork 76
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
feat(@desktop/wallet): Swap modal launch flows #14740
Conversation
Jenkins BuildsClick to see older builds (29)
|
8133469
to
710ec36
Compare
710ec36
to
1cfe49c
Compare
1cfe49c
to
dad4f9c
Compare
dad4f9c
to
daa652c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM aside from Global.featureFlag thing that Lukas mentioned (Would like to know as well, what is the approach we agreed on)
Please have a look at my comment about ERC1155
daa652c
to
515d1f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Probably needs a rebase to fix the e2e
tests
515d1f0
to
21f43ea
Compare
@caybro @Khushboo-dev-cpp there is a problem with Left Settings pane, my suspect is that is scrolls infinite (test wants to scroll down to the Sign out and Quit option) I recently made a commit that impacts this area (status-im/desktop-qa-automation#681) , the test was not scrolling at all on my mac I launched the tests before merging this and it was okay ish (even yesterday in nightly). I can revert the commit , but from UI perspective, did we change anything recently to this Left Pane? |
@anastasiyaig thanks for looking into this. Nothings changed on the left panel afaik, maybe @caybro knows something? |
I suspect amazing Squish. For now i will just revert my change and will look into it later properly |
@Khushboo-dev-cpp tests passed, sorry about that |
no thank you fixing :) |
fixes #14628
What does the PR do
Created a new skeleton of SwapModal which only has placeholder text for now.
Added new Qml file called SwapFormData.qml which will be the interface to fill in swap modal before its launched from anywhere in the app.
Also added approach to only share account index, network chainid and token keys in order to indentify any of these inputs.
Affected areas
SwapModal
StatusQ checklist
Screenshot of functionality (including design for comparison)
Screen.Recording.2024-05-13.at.7.25.36.PM.mov
Screen.Recording.2024-05-13.at.7.26.36.PM.mov