Skip to content
This repository has been archived by the owner on Mar 15, 2023. It is now read-only.

Near watcher #34

Merged
merged 6 commits into from Jan 5, 2023
Merged

Near watcher #34

merged 6 commits into from Jan 5, 2023

Conversation

heyitaki
Copy link
Contributor

@heyitaki heyitaki commented Dec 29, 2022

This PR adds a watcher for Near as well as a one-off backfilling script. Any data older than 5 epochs (~2.5 days) must hit an archival node to get data, so the backfill script uses a provider that hits an archive node whereas the watcher uses a regular provider.

Resolves #6 and #35

@evan-gray
Copy link
Contributor

For backfilling, I think it would be far easier to grab the transactions from https://explorer.near.org/accounts/contract.wormhole_crypto.near and backfill them in a one of script.

@heyitaki
Copy link
Contributor Author

Ok, I think that's a great idea, happy to proceed with that approach

jumpsiegel
jumpsiegel previously approved these changes Dec 29, 2022
Copy link
Contributor

@evan-gray evan-gray left a comment

Choose a reason for hiding this comment

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

Mostly questions 🙃

watcher/package.json Outdated Show resolved Hide resolved
watcher/src/consts.ts Outdated Show resolved Hide resolved
watcher/src/types/near.ts Show resolved Hide resolved
watcher/src/types/near.ts Outdated Show resolved Hide resolved
watcher/scripts/backfillNear.ts Outdated Show resolved Hide resolved
watcher/scripts/backfillNear.ts Outdated Show resolved Hide resolved
watcher/src/watchers/NearWatcher.ts Show resolved Hide resolved
jumpsiegel
jumpsiegel previously approved these changes Jan 2, 2023
Copy link
Contributor

@evan-gray evan-gray left a comment

Choose a reason for hiding this comment

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

attempting to run the backfill resulted in a 429 and crash ☹️

@evan-gray evan-gray merged commit 8a6a237 into main Jan 5, 2023
@evan-gray evan-gray deleted the near branch January 5, 2023 16:45
@aadam-10 aadam-10 linked an issue Mar 15, 2023 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

near: ensure execution outcomes are finalized Near Support
4 participants