Skip to content

feat: pass namespace ID to DA layers#492

Merged
tzdybal merged 1 commit intomainfrom
tzdybal/single_namespace_id
Aug 17, 2022
Merged

feat: pass namespace ID to DA layers#492
tzdybal merged 1 commit intomainfrom
tzdybal/single_namespace_id

Conversation

@tzdybal
Copy link
Copy Markdown
Contributor

@tzdybal tzdybal commented Aug 17, 2022

NamespaceID is removed from Celestia DA configuration. Instead, it's added to DA interface. Mocks simply ignore this.

Resolves #425.

@tzdybal tzdybal added T:enhancement T:dev-usability-and-ux Type: Developer Usability and UX labels Aug 17, 2022
@tzdybal tzdybal self-assigned this Aug 17, 2022
@tzdybal tzdybal requested a review from YazzyYaz August 17, 2022 09:30
Copy link
Copy Markdown
Contributor

@adlerjohn adlerjohn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@tzdybal tzdybal merged commit 8b186f9 into main Aug 17, 2022
@tzdybal tzdybal deleted the tzdybal/single_namespace_id branch August 17, 2022 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T:dev-usability-and-ux Type: Developer Usability and UX T:enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Celestia DA - namespace id

2 participants