Ryan

TODO: From: @walldiss - [X] Nit: here and in other places there is no need to start error message with "failure..." since it is already an error type and will...

Removed blockstore so it can go into a new PR and can be reviewed more thoroughly, this PR is too big rn and the main part of it can get...

@Wondertan Closing because celestiaorg/dagstore#1 is merged

+1 for having this observability, the lazy invoking has caused a few wild goose chases at https://github.com/celestiaorg/celestia-node

To use SubmitTx programmatically, you ofc have to import tons of cosmossdk/app stuff, so at that point it doesn’t really make sense to use node at all…. I think it...

@Bidon15 is asking a few teams and will report back to me, but I am already confident that this endpoint is useless and would already prepare the PR in advance.

Alright, the optional params PR for `go-jsonrpc` isn’t really workable. It makes all parameters with pointers optional, and it doesn’t solve the problem that the generated client can’t actually mark...

I don't know why our API should be strictly adhering to golang semantics. If people want to use the golang errors, just import node and call the methods on the...

I want to butt in before this error is closed, and say that I find it extremely unintuitive that an error is returned when no blobs are found on the...

Good idea 👍🏼 I'm on board