Closed zakir-code closed 1 week ago
The changes in the pull request primarily involve modifications to the app/app.go
file. A new legacy
codec package is introduced and integrated into the App
structure, with the legacyAmino
variable assigned to legacy.Cdc
. The New
function is updated to create and register the legacyAmino
codec with various interfaces. Additionally, the setAnteHandler
method is revised to incorporate a new anteOptions
structure that utilizes the legacyAmino
codec for transaction handling, while maintaining the existing application structure and flow.
File | Change Summary |
---|---|
app/app.go | Added legacyAmino *codec.LegacyAmino to App struct. Updated New function to assign legacy.Cdc to legacyAmino . Revised setAnteHandler to include anteOptions using legacyAmino codec for transaction handling. |
sequenceDiagram
participant App
participant LegacyCodec
participant AnteHandler
App->>LegacyCodec: Initialize legacyAmino
App->>App: Register legacyAmino with interfaces
App->>AnteHandler: Set ante handler with anteOptions
AnteHandler->>App: Validate options
🐰 In the code, a legacy we weave,
WithlegacyAmino
, we believe.
Transactions flow, smooth and bright,
In the app's heart, a new delight!
Hopping forward, we embrace the change,
In the world of code, we're free to range! 🌟
app/app.go (3)
`25-25`: **LGTM: Import for legacy codec added** The addition of the legacy codec import aligns with the PR objective of updating to a global amino module. --- `121-122`: **LGTM: LegacyAmino field added to App struct** The addition of the `legacyAmino` field to the App struct is consistent with the PR objective of updating to a global amino module. --- `121-122`: **LGTM: Global legacy codec initialized** The assignment of `legacyAmino` to `legacy.Cdc` sets up the global legacy codec, which is consistent with the PR objective.
Summary by CodeRabbit