Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

Add Forked Changelog#65

Merged
philipsu522 merged 3 commits intomainfrom
FORKED_CHANGELOG
Oct 25, 2022
Merged

Add Forked Changelog#65
philipsu522 merged 3 commits intomainfrom
FORKED_CHANGELOG

Conversation

@philipsu522
Copy link
Contributor

Describe your changes and provide context

Create FORKED_CHANGELOG containing changes made on forked project

Testing performed to validate your change

* (x/accesscontrol)[\#33](https://github.com/sei-protocol/sei-cosmos/pull/33) Add gov proposal handler for x/accesscontrol
* (x/accesscontrol)[\#41](https://github.com/sei-protocol/sei-cosmos/pull/41) Add ante dependency decorator to define dependencies
* [\#58](https://github.com/sei-protocol/sei-cosmos/pull/58) Lazy deposits all module accounts during EndBlock for parallel DeliverTx
* (x/accesscontrol)[\#59](https://github.com/sei-protocol/sei-cosmos/pull/59) Add gov proposal type for wasm dependency mapping updates
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't want to include wasm stuff for audit right? Or are we including the basic implementation that Tony's testing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we can! @codchen @udpatil can you point me to PRs?

Copy link
Contributor

Choose a reason for hiding this comment

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

#53
This is where we added the wasm dependency stuff to ACL keeper, but the dynamic dependency usage is actually in sei-chain in the aclmapping dir

### Features
* [\#17](https://github.com/sei-protocol/sei-cosmos/pull/17) Support SR25519 algorithm for client transaction signing
* [\#23](https://github.com/sei-protocol/sei-cosmos/pull/23) Add priority to CheckTx based on gas fees
* (x/accesscontrol)[\#24](https://github.com/sei-protocol/sei-cosmos/pull/24) Add AccessControl module
Copy link
Contributor

Choose a reason for hiding this comment

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

We should also include the change that adds the dependency dag to access control, right? It initially was put in sei chain and then refactored to sei-cosmos because it fits better in the access control module

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@udpatil can you point me to the PR?

Copy link
Contributor

Choose a reason for hiding this comment

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

#36

* (x/accesscontrol)[\#30](https://github.com/sei-protocol/sei-cosmos/pull/30) Add resource hierarchy helper to build resource dependencies
* (x/accesscontrol)[\#33](https://github.com/sei-protocol/sei-cosmos/pull/33) Add gov proposal handler for x/accesscontrol
* (x/accesscontrol)[\#41](https://github.com/sei-protocol/sei-cosmos/pull/41) Add ante dependency decorator to define dependencies
* [\#58](https://github.com/sei-protocol/sei-cosmos/pull/58) Lazy deposits all module accounts during EndBlock for parallel DeliverTx
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we also wait for these PRs as it also changes the bank logic:

#61
#63

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added

@philipsu522 philipsu522 merged commit 2a47756 into main Oct 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants