-
Notifications
You must be signed in to change notification settings - Fork 79
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
Script for DKIM Reg #135
Script for DKIM Reg #135
Conversation
saleel
commented
Nov 6, 2023
•
edited
Loading
edited
- Add scripts package.
- Modify dkim key script to work with new DKIMRegistry.sol (Use poseidon_hash(pubkey))
- Remove defaults from DKIMregistry.sol
- Add option for setting multiple keys for a domain in one tx in DKIMregistry.sol
- Remove MailServer.sol
@@ -0,0 +1,21 @@ | |||
{ | |||
"name": "@zk-email/scripts", |
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.
this should just go in helpers imo?
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.
or this dkim selector archive site can be its own site
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.
this should just go in helpers imo?
I thought of it, but helpers is a sdk that other people would be importing to their code. We don't want these scripts to be part of it right.
Typescript errors for me: And I also don't see |
They will be generated when you run |
ce7cb14
to
f2284c3
Compare
New dependencies detected. Learn more about Socket for GitHub ↗︎
|