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

Operation status flow in Massa wallet #730

Open
loonggamma opened this issue Sep 15, 2023 · 4 comments
Open

Operation status flow in Massa wallet #730

loonggamma opened this issue Sep 15, 2023 · 4 comments

Comments

@loonggamma
Copy link
Contributor

loonggamma commented Sep 15, 2023

Context:
Task follows from discussion here: #456
Currently, we can't show full history of operations per wallet account. For Mainnet, we must at least have the proper flow that comes after user initiates operations from the wallet UI.

At the moment user can initiate:
a) transaction operations (sending MAS to other address)
b) upload website operations (batch of 3+ operations signed at once)

User flow:

  1. User enters all information to initiate Operations in the wallet UI
  2. User successfully signs operations in the wails popup
  3. User sees the status of operation. Statuses are: pending, success, failed.

How-to:
For transactions:
Pending status: - operation is in not final and not stale block.
Success status: - operation is finalized.
For SC operation types: operation's execution must be 'success' as well.
Failed status: - operation failed for multiple reasons.

Reasons for failure must be inspected. Some examples:

  • not enough MAS for gas
  • operation in a stale block (not finalized)
  • execution stopped (for SC operation types)
  • other to identify

Data to show (in the first version) @LucasF243:

  • op ID (i'd link the explorer's Operation page here)
  • op type
  • from:to
  • value
  • fees

Technical details:
tbd

Q/A steps:
tbd

@loonggamma
Copy link
Contributor Author

@peterjah @Thykof what other failure reasons for Operations can we identify?

@loonggamma loonggamma added P2 and removed P1 labels Sep 22, 2023
@pivilartisant
Copy link
Contributor

cc @mazmassa wdyt ?

@loonggamma
Copy link
Contributor Author

loonggamma commented Sep 25, 2023

Not finalized yet. We had technical discussions which are being implemented in the designs. I will ping you to validate when ready.

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

No branches or pull requests

2 participants