<!-- < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < ☺ v ✰ Thanks for opening an issue! ✰ v Before smashing the submit button please review the template. v Word of caution: poorly thought-out proposals may be rejected v without deliberation ☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --> ## Summary Integrate Textual with the existing sign modes. Part of #11970 ## Problem Definition Textual work has been happening in a separate package `tx/textual`. We should wire it up to the existing sign mode handlers. ## Proposal - [ ] Add new new sign mode handler (see previous work #12077) - the `signModeTextualHandler` struct should hold a `Textual` field - [ ] Modify the [`NewTxConfig`](https://github.com/cosmos/cosmos-sdk/blob/6f070623741fe0d6851d79ada41e6e2b1c67e236/x/auth/tx/config.go#L28) function signature for Textual, or a new function `NewTxConfigWithTextual` - [ ] Textual needs access to the following parameters: 1. `bodyBz` and `authInfoBz`: easily fetchable from the `wrapper` 2. `context.Context`: easy once https://github.com/cosmos/cosmos-sdk/issues/13646 is done 3. `coinMetadataQuerier`: to be set in `app.go` (for manual wiring) or tx module (for depinject). It needs to use the bankKeeper. - [ ] Find a good public-facing API
Summary
Integrate Textual with the existing sign modes.
Part of #11970
Problem Definition
Textual work has been happening in a separate package
tx/textual. We should wire it up to the existing sign mode handlers.Proposal
signModeTextualHandlerstruct should hold aTextualfieldNewTxConfigfunction signature for Textual, or a new functionNewTxConfigWithTextualbodyBzandauthInfoBz: easily fetchable from thewrappercontext.Context: easy once SignModeHandler'sGetSignBytes: add Context arg #13646 is donecoinMetadataQuerier: to be set inapp.go(for manual wiring) or tx module (for depinject). It needs to use the bankKeeper.