-
Notifications
You must be signed in to change notification settings - Fork 329
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove account dual dispatchers #1168
Remove account dual dispatchers #1168
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## remove-dual-dispatchers #1168 +/- ##
==========================================================
Coverage ? 89.66%
==========================================================
Files ? 53
Lines ? 1316
Branches ? 0
==========================================================
Hits ? 1180
Misses ? 136
Partials ? 0
Continue to review full report in Codecov by Sentry.
|
There are also |
Those are removed in #1175 |
…ro-contracts into feat/remove-account-dual-dispatchers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
b8a3ed5
into
OpenZeppelin:remove-dual-dispatchers
* feat: remove access dual dispatchers (#1154) * feat: remove account dual dispatchers (#1168) * Remove token dual dispatchers (#1175) * feat: remove modules * fix: mock * fix: linter * fix: tests * fix: mock * feat: apply review suggestions * feat: fix import * Remove dual dispatchers docs and CHANGELOG (#1177) * feat: update docs * feat: update CHANGELOG * fix: typo * fix: mod * feat: remove unused imports * fix: README * feat: remove selectors * Update docs/modules/ROOT/pages/interfaces.adoc Co-authored-by: Andrew Fleming <[email protected]> * feat: add mention and link to dispatcher types * feat: remove import * Update docs/modules/ROOT/pages/interfaces.adoc Co-authored-by: Andrew Fleming <[email protected]> * fix: README --------- Co-authored-by: Andrew Fleming <[email protected]> --------- Co-authored-by: Andrew Fleming <[email protected]>
Partially fixes #1128