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 registrar to use expiration instead of insertion time #40
base: master
Are you sure you want to change the base?
Conversation
"dns-packet": "^5.0.4", | ||
"eth-ens-namehash": "^2.0.8", | ||
"js-sha3": "^0.5.7", | ||
"web3": "1.0.0-beta.34" |
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.
web3js is in use at
dnsregistrar/lib/dnsregistrar.js
Line 3 in c3fde64
const Web3 = require('web3'); |
dnsregistrar.js
file all together
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.
Please update README to remove reference to DNSRegistrarJs
PTAL |
See also ensdomains/dnssec-oracle#162 and ensdomains/dnssecoraclejs#6 |
@@ -1,23 +1,17 @@ | |||
{ | |||
"name": "@ensdomains/dnsregistrar", | |||
"main": "dist/dnsregistrar.js", |
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.
Can you get rid of dist/dnsregistrar.js
and replace it with something like https://github.com/ensdomains/ens/blob/master/index.js ?
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.
I've removed the main reference. There doesn't seem like there's much point in adding that given our pending change to a monorepo.
PTAL: |
No description provided.