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

Retrieve from address from txHash #404

Closed
3 tasks done
hernandiegosavio opened this issue Jun 13, 2023 · 0 comments · Fixed by #408
Closed
3 tasks done

Retrieve from address from txHash #404

hernandiegosavio opened this issue Jun 13, 2023 · 0 comments · Fixed by #408
Assignees
Milestone

Comments

@hernandiegosavio
Copy link
Collaborator

hernandiegosavio commented Jun 13, 2023

Currently we dont have information regarding the from address or wallet that created a VAA

We need to include a component that based on the origin chain and the tx hash connects to the rpc node and retreive the from address.

Persistence should be added to the VAA collection in order to filter that data in the future.

Tasks

  1. v1
    agodnic
@hernandiegosavio hernandiegosavio added this to the v1 milestone Jun 13, 2023
@gipsh gipsh changed the title Data missing from the frontend Retrieve from address from txHash Jun 14, 2023
@agodnic agodnic self-assigned this Jun 14, 2023
@agodnic agodnic linked a pull request Jun 14, 2023 that will close this issue
agodnic added a commit that referenced this issue Jun 14, 2023
### Summary

Tracking issue: #404

The WormholeScan UI needs to display the sender address for each token bridge VAA. This pull request modifies the `tx-tracker` service to obtain that information for Solana and eight EVM chains.

The transaction sender will become accessible through the following endpoints:
* `GET /api/v1/global-tx/{chain}/{emitter}/{seq}`: field `originTx.from`.
* `GET /api/v1/transactions`: field `originAddress`.
In both cases, the field is nullable (i.e.: sometimes it may not be available due to eventual consistency or internal errors)
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

Successfully merging a pull request may close this issue.

2 participants