Skip to content

feat: asset vault sync#1176

Merged
bobbinth merged 60 commits intonextfrom
igamigo-sync-assets
Aug 29, 2025
Merged

feat: asset vault sync#1176
bobbinth merged 60 commits intonextfrom
igamigo-sync-assets

Conversation

@igamigo
Copy link
Copy Markdown
Collaborator

@igamigo igamigo commented Aug 21, 2025

Closes #1099

@igamigo igamigo force-pushed the igamigo-sync-assets branch 2 times, most recently from 7f9c2f0 to 2b50506 Compare August 22, 2025 20:44
@igamigo igamigo marked this pull request as ready for review August 25, 2025 02:43
@igamigo igamigo force-pushed the igamigo-sync-assets branch from aefa8d6 to 742c864 Compare August 27, 2025 19:53
@igamigo igamigo requested a review from bobbinth August 28, 2025 13:02
Comment thread crates/block-producer/src/server/mod.rs Outdated
let store = StoreClient::new(self.store_url.clone());

// Retry fetching the chain tip from the store until it succeeds.
// retry fetching the chain tip from the store until it succeeds.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nit: bit of a random change, could probably remain upper case

Copy link
Copy Markdown
Collaborator Author

@igamigo igamigo Aug 28, 2025

Choose a reason for hiding this comment

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

Yeah, not even sure why/how I did this, I think I looked at this code to review that PR related to returning the genesis commitment on the status endpoint

Comment thread crates/rpc/README.md
Comment thread crates/store/src/server/rpc_api.rs
Comment thread crates/store/src/db/models/queries/accounts.rs Outdated
Base automatically changed from igamigo-map-sync-rpc to next August 29, 2025 04:52
Copy link
Copy Markdown
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

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

All looks good! Thank you!

@bobbinth bobbinth merged commit 99c9848 into next Aug 29, 2025
6 checks passed
@bobbinth bobbinth deleted the igamigo-sync-assets branch August 29, 2025 05:15
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.

Implement account vault sync endpoint

3 participants