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

Bing/ft watcher worker #339

Merged
merged 2 commits into from
Aug 5, 2024
Merged

Bing/ft watcher worker #339

merged 2 commits into from
Aug 5, 2024

Conversation

bingyuyap
Copy link
Contributor

@bingyuyap bingyuyap commented Jul 26, 2024

This PR updates the watcher code such that ft_watcher can be run

main changes:

  • add ft to Mode
  • add INITIAL_DEPLOYMENT_BLOCK for FT and add a reusable type called NetworkChainBlockMapping
  • add map to map INITIAL_DEPLOYMENT_BLOCK to their modes
  • add map to map collection name to their modes
  • instead of using isNTT boolean use Mode
  • update to some naming of classes and methods

db updates:

  • fast_vaa_id and fast_vaa_hash to be UNIQUE as at some point in time they might be null

@bingyuyap bingyuyap force-pushed the bing/ft_watcher_worker branch 4 times, most recently from b1361c1 to 72afc77 Compare July 26, 2024 16:32
@bingyuyap bingyuyap requested review from panoel and evan-gray and removed request for panoel July 26, 2024 17:01
@bingyuyap bingyuyap marked this pull request as ready for review July 26, 2024 17:01
@bingyuyap bingyuyap requested a review from panoel July 26, 2024 17:02
panoel
panoel previously approved these changes Jul 31, 2024
@bingyuyap bingyuyap force-pushed the bing/ft_watcher_worker branch 2 times, most recently from 48e67f5 to 67c3d97 Compare July 31, 2024 17:34
@bingyuyap bingyuyap requested a review from panoel July 31, 2024 17:44
@evan-gray evan-gray force-pushed the bing/ft_watcher_worker branch from 67c3d97 to ece11a5 Compare August 4, 2024 02:26
@bingyuyap bingyuyap merged commit 1f4fabc into main Aug 5, 2024
3 checks passed
@bingyuyap bingyuyap deleted the bing/ft_watcher_worker branch August 5, 2024 16:47
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.

3 participants