Skip to content

feat: add governance to set_max_supply, max_supply_updated event, metadata_uri validation, and vitest configs - #368

Merged
DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
blurbeast:feat/supply-cap-metadata-governance-vitest
Aug 30, 2026
Merged

feat: add governance to set_max_supply, max_supply_updated event, metadata_uri validation, and vitest configs#368
DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
blurbeast:feat/supply-cap-metadata-governance-vitest

Conversation

@blurbeast

Copy link
Copy Markdown
Contributor

Closes #181
Closes #182
Closes #184
Closes #186

this pr resolves issues raised in the issues tagged above

…adata_uri validation, and vitest configs

- Validate that metadata_uri in credential-nft mint is non-empty, >= 8 chars, and well-formed
- Add max_supply_updated event to learn-token set_max_supply
- Implement governance safeguard (max 2x increase limit) on set_max_supply
- Add vitest.config.ts for chainlearn-api and chainlearn-indexer
- Fix duplicate function definitions in storage and compiler warnings

Closes ChainLearnOfficial#181, Closes ChainLearnOfficial#182, Closes ChainLearnOfficial#184, Closes ChainLearnOfficial#186
@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@blurbeast Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@DeFiVC
DeFiVC merged commit 3b9b199 into ChainLearnOfficial:main Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants