-
Notifications
You must be signed in to change notification settings - Fork 264
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
chore: backport tx status #1405
chore: backport tx status #1405
Conversation
1d2e339
to
0f61c93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
first pass looks good to me 👍
Is there an issue for this or more context?
Co-authored-by: CHAMI Rachid <chamirachid1@gmail.com>
Thanks for the review! Here are the links to the work I'm backporting from main: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utAck.
Awesome stuff 🔥 Left some unblocking nits.
Also, would be good to update the openapi specs to reflect these changes both in main in this branch. Can be handled in a separate PR or an issue for this to be done later.
@@ -17,6 +17,13 @@ import ( | |||
"github.com/tendermint/tendermint/types" | |||
) | |||
|
|||
const ( | |||
txStatusUnknown string = "UNKNOWN" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[non-blocking][microscopic nit]
probably document the cases where the status is unknown
Height int64 `json:"height"` | ||
Index uint32 `json:"index"` | ||
ExecutionCode uint32 `json:"execution_code"` | ||
Status string `json:"status"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[non blocking]
maybe document these
Opened an issue #1406 |
b1d52fd
into
celestiaorg:v0.34.x-celestia
…tiaorg#1405) (celestiaorg#1407) * doc: improve documentation of BlockParams.MaxBytes (manual backport of celestiaorg#1405) * Applying @sergio-mena suggestion Co-authored-by: Sergio Mena <sergio@informal.systems> * Applying @jmalicevic suggestion Co-authored-by: Jasmina Malicevic <jasmina.dustinac@gmail.com> --------- Co-authored-by: Sergio Mena <sergio@informal.systems> Co-authored-by: Jasmina Malicevic <jasmina.dustinac@gmail.com>
Description
backport TxStatus work from main