-
Notifications
You must be signed in to change notification settings - Fork 120
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
feat: support ethda protocol #137
base: develop
Are you sure you want to change the base?
Conversation
4b4dd5f
to
3da553a
Compare
c357d0e
to
7b1ea94
Compare
What is ethda? can you elaborate a full description of the PR? |
gm! After a team discussion we're reconsidering the idea of adding the different DA protocol integrations directly on this repo. Instead we'd like this code to live on a repo you fully own, and for it to be imported here as an external dependency. In a nutshell, this PR should only contain the modifications done on:
Anything else should be handled on a repo of yours. Sorry for the confusion, and the last minute change of mind, but we believe this will be better for everyone |
Hi arnaubennassar & vcastellm, sorry for late reply. EthDA by itself is an Ethereum zk rollup built with Polygon CDK, and is targeted to sever as an Alt-DA solution for other CDK validiums, just as Celestia or Avail does. You could find more info in our website. This PR is to support starting a CDK validium with EthDA as DA layer (instead of DAC nodes). And thanks for your illustration, we will continue to update the PR following your guidelines. |
8033805
to
dc5c32f
Compare
@arnaubennassar Hi arnaubennassar, thank you for your reply. We have extracted the EthDA-related code into a separate library. The current changes now only involve |
d13357f
to
09530a6
Compare
09530a6
to
b2f0a14
Compare
b2f0a14
to
f280bf2
Compare
ethda website: https://ethda.io/