Skip to content
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

V0.50.1 patched #438

Closed
wants to merge 29 commits into from
Closed

V0.50.1 patched #438

wants to merge 29 commits into from

Conversation

yan-soon
Copy link

@yan-soon yan-soon commented Dec 7, 2023

key changes:

  • revert new cosmos-sdk change where multiSend can only have one sender (Make MsgMultiSend single sender cosmos/cosmos-sdk#12601)
  • revert new cosmos-sdk change where send and multiSend typeAsserts the bank keeper to be of type BaseKeeper (this type assertion has issues with our implementation as our bank keeper is customised due to the x/alliance module, where the keeper is a wrapper of the original x/bank BaseKeeper, resulting in the type assertion check failing
    102b3f2

@RavenXce
Copy link
Member

reverting msgmultisend is not safe

baseapp/abci.go Outdated Show resolved Hide resolved
client/pruning/main.go Outdated Show resolved Hide resolved
@@ -572,9 +578,9 @@ func (mr *MockBankKeeperMockRecorder) InitGenesis(arg0, arg1 interface{}) *gomoc
}

// InputOutputCoins mocks base method.
func (m *MockBankKeeper) InputOutputCoins(ctx context.Context, input types0.Input, outputs []types0.Output) error {
func (m *MockBankKeeper) InputOutputCoins(ctx context.Context, inputs []types0.Input, outputs []types0.Output) error {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we have to change this?

x/evidence/abci.go Outdated Show resolved Hide resolved
x/distribution/types/params.go Show resolved Hide resolved
store/pruning/types/options.go Outdated Show resolved Hide resolved
store/pruning/types/options.go Outdated Show resolved Hide resolved
@@ -136,6 +140,9 @@ swagger = {{ .API.Swagger }}
# Address defines the API server to listen on.
address = "{{ .API.Address }}"

# Oracle-address defines the API server to listen on for oracle service.
oracle-address = "{{ .API.OracleAddress }}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y are there oracle stuff here?

Copy link
Author

@yan-soon yan-soon Jan 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

},
API: APIConfig{
Enable: false,
Swagger: false,
Address: DefaultAPIAddress,
OracleAddress: DefaultOracleAPIAddress,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y are there oracle stuff here?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -108,6 +111,9 @@ type APIConfig struct {
// Address defines the API server to listen on
Address string `mapstructure:"address"`

// OracleAddress defines if the API server to listen on for oracle service
OracleAddress string `mapstructure:"oracle-address"`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y are there oracle stuff here?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yan-soon yan-soon closed this Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants