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

wallet - Integrate Backend for Transaction Progress Page #18307

Closed
J-Son89 opened this issue Dec 27, 2023 · 1 comment · Fixed by #18506
Closed

wallet - Integrate Backend for Transaction Progress Page #18307

J-Son89 opened this issue Dec 27, 2023 · 1 comment · Fixed by #18506
Assignees
Labels
feature feature requests

Comments

@J-Son89
Copy link
Contributor

J-Son89 commented Dec 27, 2023

related to this page: https://www.figma.com/file/HKncH4wnDwZDAhc4AryK8Y/Wallet-for-Mobile?type=design&node-id=7701-158136&mode=design&t=tTZbzwJmyQXPAlB3-4

to integrate this, once a transaction is authorized the page is in it's initial state ("Sending...) -
Then we listen for the Status Go signal "pending-transaction-status-changed" - using this event we then make a request to "wallet_getMultiTransactions" and if there is a time stamp the block has been accepted and otherwise it has been rejected.

This is all that is needed for this issue and can be reflected in the most minimal way possible, i.e the title changing.
We can leave the advanced UI and calculations for transaction confirmed for this issue:
#16934

Data Model Notes -
As suggested by @ulisesmac, transactions should live within an account. The current model has transaction as it's own key on the wallet db model, move this into the specific account keys.

Note: Add the correct navigation into this work for closing this page. i.e it should navigate to the accounts page of that account.

@J-Son89
Copy link
Contributor Author

J-Son89 commented Jan 5, 2024

this is ready to be worked on :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature feature requests
Development

Successfully merging a pull request may close this issue.

1 participant