OpenZeppelin / OpenZeppelin/openzeppelin-contracts
Support ERC-173
Open
Nobody has claimed this yet.
breaking change
- Dominant language
- Solidity
- Stars
- 27.2k
- Forks
- 12.4k
- Avg merge
- 2d 19h
- Merged PRs (30d)
- 33
Description
Motivation
There's an open PR for supporting ERC173, and AccessControlDefaultAdminRules.sol imports IERC5313, maybe Ownable should do the same.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the ERC-173 proposal and the referenced AccessControlDefaultAdminRules.sol import, then inspect Ownable to determine whether the same interface support belongs there. Check the existing open PR mentioned in the issue and confirm the intended scope before adding coverage that demonstrates ERC-173 support.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- solidity
- Domain
- blockchain
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100