Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Use the IndexWrappedTxDecoder universally #1665

Closed
evan-forbes opened this issue Apr 26, 2023 · 0 comments · Fixed by #1666
Closed

Use the IndexWrappedTxDecoder universally #1665

evan-forbes opened this issue Apr 26, 2023 · 0 comments · Fixed by #1666
Assignees
@rootulp

Comments

@evan-forbes
Copy link
Member

atm, due to the tx config in the sdk, we can't actually replace the tx decoder globally. This has resulted in a few different bugs, including but not limited to #1620 and #485.

We should modify our encoding config to use the index wrapper tx decoder universally.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@rootulp rootulp

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: override TxDecoder rootulp/celestia-app
2 participants
@rootulp @evan-forbes