-
-
Notifications
You must be signed in to change notification settings - Fork 196
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
update addressBook with entries by chainId #152
Conversation
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.
One minor inline comment, otherwise LGTM.
Codecov Report
@@ Coverage Diff @@
## develop #152 +/- ##
======================================
Coverage 100% 100%
======================================
Files 21 21
Lines 1432 1439 +7
Branches 192 193 +1
======================================
+ Hits 1432 1439 +7
Continue to review full report at Codecov.
|
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.
update addressBook with entries by chainId add isValidEns checking for address book name
update addressBook with entries by chainId add isValidEns checking for address book name
previously the address book was only keyed by address, so a user was only allowed to have one entry per address regardless of network