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

feat: search for deposit id by chain #1122

Merged
merged 3 commits into from
Dec 22, 2023
Merged

feat: search for deposit id by chain #1122

merged 3 commits into from
Dec 22, 2023

Conversation

pxrl
Copy link
Contributor

@pxrl pxrl commented Dec 17, 2023

We can now search an arbitrary deposit id on a given chain to find all of its information. This has been very helpful to resolve deposits when looking at fills.

We can now search an arbitrary deposit id on a given chain to find all
of its information. This has been very helpful to resolve deposits when
looking at fills.
Copy link
Contributor

@mrice32 mrice32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! One minor type nit and one question

scripts/spokepool.ts Show resolved Hide resolved
scripts/spokepool.ts Show resolved Hide resolved
@pxrl pxrl merged commit 659a26f into master Dec 22, 2023
4 checks passed
@pxrl pxrl deleted the pxrl/findDeposit branch December 22, 2023 02:58
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 this pull request may close these issues.

4 participants