Skip to content
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

Integrate Sign and Approve modals into SwapModal #15443

Open
Tracked by #14786
Khushboo-dev-cpp opened this issue Jul 3, 2024 · 2 comments
Open
Tracked by #14786

Integrate Sign and Approve modals into SwapModal #15443

Khushboo-dev-cpp opened this issue Jul 3, 2024 · 2 comments
Assignees
Labels
Milestone

Comments

@Khushboo-dev-cpp
Copy link
Contributor

Khushboo-dev-cpp commented Jul 3, 2024

We need to integrate sign modal created here into swap #15410, as well as approve, created here: #15460

@noeliaSD
Copy link
Contributor

noeliaSD commented Jul 3, 2024

Ideally it shold be covered by #14785. cc: @caybro

@caybro
Copy link
Member

caybro commented Jul 3, 2024

Ideally it shold be covered by #14785. cc: @caybro

I'd rather keep it under a separate task

@caybro caybro self-assigned this Jul 3, 2024
@caybro caybro added this to the 2.30.0 Beta milestone Jul 4, 2024
caybro added a commit that referenced this issue Jul 4, 2024
- create a new UI component
- add it to StoryBook
- TODO: add QML tests
- integration will be done as part of
#15443

Fixes #15442
@caybro caybro changed the title Integrate Sign modal into Swap Integrate Sign and Approve modals into SwapModal Jul 4, 2024
caybro added a commit that referenced this issue Jul 4, 2024
- create a new UI component
- add it to StoryBook
- TODO: add QML tests
- integration will be done as part of
#15443

Fixes #15442
caybro added a commit that referenced this issue Jul 4, 2024
- create a new UI component
- add it to StoryBook
- add QML tests
- integration will be done as part of
#15443

Fixes #15442
caybro added a commit that referenced this issue Jul 4, 2024
- create a new UI component
- add it to StoryBook
- add QML tests
- integration will be done as part of
#15443

Fixes #15442
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Next iteration
Development

No branches or pull requests

3 participants