Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add a virtual _checkRole(bytes32) internal function to AccessControl #3137

Merged

Conversation

Amxx
Copy link
Collaborator

@Amxx Amxx commented Jan 26, 2022

Needed for building AccessControlCrossChain (and possibly other extensions)

PR Checklist

  • Tests
  • Changelog entry

@Amxx Amxx self-assigned this Jan 26, 2022
@Amxx Amxx requested a review from frangio January 26, 2022 16:07
@Amxx Amxx changed the title Add a virtual _onlyRole(bytes32) modifier to AccessControl Add a virtual _checkRole(bytes32) modifier to AccessControl Jan 26, 2022
@Amxx Amxx changed the title Add a virtual _checkRole(bytes32) modifier to AccessControl Add a virtual _checkRole(bytes32) internal function to AccessControl Jan 26, 2022
Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@Amxx Amxx merged commit fb950c6 into OpenZeppelin:master Jan 26, 2022
@Amxx Amxx deleted the feature/AccessControl/internalOnlyOwner branch January 26, 2022 20:36
@Amxx Amxx mentioned this pull request Jan 26, 2022
1 task
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants