Skip to content

chore: bump protocol to rc.7 + fix genesis faucet - #2538

Draft
SantiagoPittella wants to merge 2 commits into
nextfrom
santiagopittella-bump-protocol-fix-genesis
Draft

chore: bump protocol to rc.7 + fix genesis faucet#2538
SantiagoPittella wants to merge 2 commits into
nextfrom
santiagopittella-bump-protocol-fix-genesis

Conversation

@SantiagoPittella

@SantiagoPittella SantiagoPittella commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

Bumps the protocol dependencies to rc.7 + updates the genesis to use the new create_native_fungible_faucet_for_genesis function for faucet creation.

It is not mergeable until the protocol publishing works fully with all its crates.

Changelog

changelog = "none"
reason    = "Internal change only."

@igamigo igamigo left a comment

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.

Thanks!

Comment on lines +495 to +498
let fee_policy = BasicConstantFeePolicy::new().with_fees([
(MintNote::script_root(), AssetAmount::ZERO),
(BurnNote::script_root(), AssetAmount::ZERO),
]);

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.

We could also add an assertion for making sure the network account policy covers the sponsorship note as well. This should be included in the auth component by default but if for some reason it's not, then the faucet will not work. It's also not explicit because it's hidden within the constructors we used so it will help as documentation of sorts.

@igamigo

igamigo commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Would've been nice to also implement the changes related to 0xMiden/protocol#3746, but it looks like the changes will not be in this version because there is a specific release branch and the PR was merged to next (cc @mmagician)

@SantiagoPittella

Copy link
Copy Markdown
Collaborator Author

Would've been nice to also implement the changes related to 0xMiden/protocol#3746, but it looks like the changes will not be in this version because there is a specific release branch and the PR was merged to next (cc @mmagician)

If we want to, I can open an issue in the protocol with those changes. Though don't know when and if it will be released soon.

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