Skip to content
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

Decouple DAS from celestai-core #506

Closed
4 tasks done
Tracked by #491
evan-forbes opened this issue Aug 20, 2021 · 1 comment
Closed
4 tasks done
Tracked by #491

Decouple DAS from celestai-core #506

evan-forbes opened this issue Aug 20, 2021 · 1 comment
Assignees

Comments

@evan-forbes
Copy link
Member

evan-forbes commented Aug 20, 2021

Summary

Since we decided to delegate DAS to celestia-node, we should remove this functionality in celestia-core.

Details

Per ADR 007, there have been extensive changes made to incorporate DAS into our fork of tendermint. Most of these changes will need to be reverted, and what remains should either be refactors so that it doesn't import celestia-core, moved to celestia-node, or moved to it's own repo.

Action Items

References

tracking issue #491
ADR #504

@evan-forbes
Copy link
Member Author

While there is still some minor refactoring we could do (see #505), since #523 was merged, we have purged DAS from celestia-core and this issue can be closed.

@evan-forbes evan-forbes mentioned this issue Aug 31, 2021
31 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant