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.

Sign up for GitHub

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

Jump to bottom

das: implement DASer #177

Merged
merged 14 commits into from
Nov 17, 2021
Merged

das: implement DASer #177

merged 14 commits into from
Nov 17, 2021

Conversation

Wondertan
Copy link
Member

Content

  • Initial DASer implementation which just listens for headers and DASes them
  • Adds DASer and Header Service to the node.

Refs

Based on #171
Closes #166

@Wondertan Wondertan self-assigned this Nov 9, 2021
@Wondertan Wondertan mentioned this pull request Nov 9, 2021
Closed
@Wondertan Wondertan force-pushed the hlib/daser-itself branch 2 times, most recently from 2ea78c5 to b4af8aa Compare November 9, 2021 13:36
renaynay
renaynay reviewed Nov 9, 2021
View reviewed changes
das/daser.go Outdated Show resolved Hide resolved
das/daser.go Show resolved Hide resolved
node/services/service.go Show resolved Hide resolved
@renaynay
Copy link
Member

On hold while we add tests.

@Wondertan Wondertan force-pushed the hlib/daser-itself branch 2 times, most recently from 587b2e0 to 86f734b Compare November 10, 2021 15:51
@evan-forbes evan-forbes mentioned this pull request Nov 12, 2021
Merged
renaynay
renaynay previously approved these changes Nov 17, 2021
View reviewed changes
mattdf
mattdf reviewed Nov 17, 2021
View reviewed changes
das/daser.go Show resolved Hide resolved
@Wondertan Wondertan merged commit 4d02ef5 into main Nov 17, 2021
@Wondertan Wondertan deleted the hlib/daser-itself branch November 17, 2021 10:22
@Wondertan Wondertan mentioned this pull request Nov 24, 2021
Closed
@Wondertan Wondertan restored the hlib/daser-itself branch January 7, 2022 14:40
@Wondertan Wondertan deleted the hlib/daser-itself branch January 7, 2022 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mattdf mattdf mattdf approved these changes

@renaynay renaynay renaynay approved these changes

@liamsi liamsi Awaiting requested review from liamsi

Assignees

@Wondertan Wondertan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

das: DASer
3 participants
@Wondertan @renaynay @mattdf