-
Notifications
You must be signed in to change notification settings - Fork 57
Update tx.rs to include return_code in server respone #141
Conversation
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.
Looks good to me but I would keep the comments ^^. They are not the best comments but otherwise we forget some important things.
src/server/tx.rs
Outdated
@@ -189,9 +182,10 @@ impl TryFrom<Row> for TxInfo { | |||
let gas_limit_multiplier = row.try_get("gas_limit_multiplier")?; | |||
let code: Option<Vec<u8>> = row.try_get("code")?; | |||
let data: Option<Vec<u8>> = row.try_get("data")?; | |||
let return_code = row.try_get("return_code").ok(); // Parsing return_code |
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.
Maybe we could use row.try_get("return_code")?;
to keep it align with the rest.
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.
yes sure
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.
If you can do the change I think we will be good to merge.
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.
Added comments back.. sorry for removing it
changed to row.try_get("return_code")?;
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.
updated the row.try_get("return_code")?;
No description provided.