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.

Sign up for GitHub

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

Jump to bottom

Bing/ft watcher mainnet #365

Merged
merged 1 commit into from
Aug 29, 2024
Merged

Bing/ft watcher mainnet #365

merged 1 commit into from
Aug 29, 2024

Conversation

bingyuyap
Copy link
Contributor

@bingyuyap bingyuyap commented Aug 27, 2024 •
edited
Loading

This PR adds mainnet related configuration and logic for the Fast Transfer Watcher:

Config

  • Config for Solana, Base and Arbitrum
    • blocks to start observing from on Mainnet
    • Mainnet contract addresses
  • Update example-liquidity-layer-solana tarball file to have Mainnet program ID for Matching Eng

Logic

  • Add conditional function selector for Mainnet vs Testnet. This is because on Mainnet the entry point is the Swap Layer but on Testnet it is the Token Router
  • Add logic to handle missing fast_vaa_hash in Mainnet and Testnet.

Clean up

  • Fix makeBlockKey on EVM
  • Improve logging on FTSolanaWatcher
  • Clean up makeFinalizedFTWatcher

bingyuyap
bingyuyap commented Aug 28, 2024
View reviewed changes
watcher/package.json Outdated Show resolved Hide resolved
bingyuyap
bingyuyap commented Aug 28, 2024
View reviewed changes
watcher/src/index.ts Outdated Show resolved Hide resolved
@bingyuyap bingyuyap requested a review from panoel August 29, 2024 00:42
@bingyuyap bingyuyap marked this pull request as ready for review August 29, 2024 00:47
panoel
panoel previously approved these changes Aug 29, 2024
View reviewed changes
@bingyuyap
ft_watcher: update logs
056557c 
ft_watcher: this works
ft_watcher: add mainnet solana addresses
ft_watcher: add conditional to detect mainnet function selector
ft_watcher: watcher runners update
ft_watcher: update sdk const
ft_watcher: update package-lock.json
ft_watcher: handle missing vaa hash cases in testnet
ft_watcher: fix evm time
ft_watcher: update reference
ft_watcher: clean up
ft_watcher: update package-lock.json
ft_watcher: plz fix ci
ft_watcher: remove script
ft_watcher: clean up
Signed-off-by: bingyuyap <[email protected]>
@bingyuyap bingyuyap requested a review from panoel August 29, 2024 15:45
@bingyuyap bingyuyap merged commit 87903ab into main Aug 29, 2024
3 checks passed
@bingyuyap bingyuyap deleted the bing/ft_watcher_mainnet branch August 29, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@panoel panoel panoel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bingyuyap @panoel