-
Notifications
You must be signed in to change notification settings - Fork 252
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: register blob msg server in module #1653
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1653 +/- ##
=======================================
Coverage 50.98% 50.99%
=======================================
Files 92 92
Lines 5749 5748 -1
=======================================
Hits 2931 2931
+ Misses 2519 2518 -1
Partials 299 299
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
Nice catch. How did it work without having the querier registered?
The querier was registered. The msg server is just for transactions |
Aaah I see, then how are we sending pay for blob transactions? |
the sdk was using the legacy mechanism, which we're also registering |
Overview
This is pretty standard to do in the sdk, not sure how we haven't done this yet given over two years of running the state machine and audits lol
Everything was still occurring via the legacy sdk.Router, but still...
Checklist