Add ERC-165 detection for ERC6909TokenSupply and ERC6909Metadata#6247
Add ERC-165 detection for ERC6909TokenSupply and ERC6909Metadata#6247ernestognw merged 4 commits intoOpenZeppelin:masterfrom
ERC6909TokenSupply and ERC6909Metadata#6247Conversation
|
| Name | Type |
|---|---|
| openzeppelin-solidity | Patch |
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
ERC6909TokenSupply and ERC6909Metadata
ERC6909TokenSupply and ERC6909MetadataERC6909TokenSupply and ERC6909Metadata
WalkthroughThis pull request adds ERC165 interface detection support to two ERC6909 extensions. The ERC6909Metadata and ERC6909TokenSupply contracts now implement Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Comment |
…6247) Co-authored-by: ernestognw <ernestognw@gmail.com> Signed-off-by: Hadrien Croubois <hadrien.croubois@gmail.com>
ERC-6909 defines three extensions which we implement. These extensions have their own intefaces but we didn't add ERC-165 detection for these interfaces.
#6246 flagged this issue and fixed it for
ERC6909ContentURI. This PR fixes it forERC6909TokenSupplyandERC6909Metadata.We will keep the fix for
ERC6909ContentURIseperate to ensure credit is given properly. There will only be one changeset for all the changes which will be part of this PR.PR Checklist
npx changeset add)