-
Notifications
You must be signed in to change notification settings - Fork 94
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
feat(tx-history): handle encoded transaction values #2133
Conversation
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
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 just one note
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.
suggestion
e45f707
to
38d779c
Compare
Signed-off-by: onur-ozkan <work@onurozkan.dev>
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.
great work !
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!
* dev: fix(indexeddb): window usage in worker env (KomodoPlatform#2131) feat(tx-history): handle encoded transaction values (KomodoPlatform#2133) fix(core): tendermint withdraws on hd accounts (KomodoPlatform#2130) fix(core): improve validation rules for table names (KomodoPlatform#2123) fix(test): improve log wait condition to fix taker restart test (KomodoPlatform#2125)
For cosmos based networks, transaction values can be base64 encoded. Previously, tx-history implementation was not capable of handling these and that leads to missing transactions in the history table. This PR fixes that.
Blocker for #2128