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: ✨ Apply Forest root changes to Buckets in Blockchain Service #335

Open
wants to merge 50 commits into
base: main
Choose a base branch
from

Conversation

ffarall
Copy link
Contributor

@ffarall ffarall commented Jan 22, 2025

This PR depends on #312.

In this PR:

  1. Handle MutationsApplied event in apply_forest_root_changes of the Blockchain Service, to apply forest root changes of MSPs.
  2. Remove insert_files_metadata call from msp_upload_file, now that this is done in the Blockchain Service.
  3. Add query_buckets_for_msp new runtime API to Providers pallet.
  4. Add event_info to generic_apply_delta function from ProofsDealerInterface. The intention is that users of this interface can pass a Vec<u8> of encoded info for context on what the delta is applied on. For the purpose of this PR, that is used for the File System pallet to pass down the encoded Bucket ID.

ffarall added 30 commits January 3, 2025 16:32
@ffarall ffarall marked this pull request as ready for review January 24, 2025 02:52
@santikaplan santikaplan requested a review from snowmead January 24, 2025 15:55
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.

1 participant