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

Fix/savings app fixes #314

Closed
wants to merge 879 commits into from
Closed

Fix/savings app fixes #314

wants to merge 879 commits into from

Conversation

Kayanski
Copy link
Contributor

@Kayanski Kayanski commented Jan 19, 2024

This PR aims at providing necessary tooling for the abstract:savings-app.

Detailed changes :

  1. Added authz querier + fee granter change (Feature/add authz wrapping #305)
  2. Authz queries implementation + tests (to know whether authz was granted)
  3. Change the wasm getter implementation to get the right artifact when deploying the dex adapter. (See this comment : 555f279#comments)

adairrr and others added 30 commits August 7, 2023 08:41
Add archway constantine-3 and archway-1 networks
Wasn't sure if it should be CONSTANTINE_3 for testnet or `networks::ARCHWAY_1` for mainnet. Opted for mainnet.

Not sure if there needs to be the "To interact with contracts on Archway..." bit on ln 8 either
Please check Kuji Mainnet - not sure if KAIYO_1 is correct.
Didn't put the network in this one
Kayanski and others added 27 commits January 24, 2024 13:58
* Added mock state env id

* Format

* Added environment info trait instead of deploy_details

* Removed deploy_details in state template impl
* fix cw-plus deps

* switch to the crates-io version
* Added cw-multi-test 0.20.0

* Removed secp, removed ibc-relayer-types

* Removed ibc-relayer-types completely

* updated prost typesé

* Start testing authz

* Added authz test

* add authz querier

* tx handler error with anyhow test

* format

* Added authz builder method for options

* Some nits

* Revert with_authz, added sender options setter

* Check rest of authz queries

* Added fee grant in daemon builder and interface

* Refactor daemon for simplified re-builderé

* Modifed async daemon as well

* Corrected SenderBuilder structure

* Clippied

* Add doc-comments

* fmt

---------

Co-authored-by: Buckram <buckram123@gmail.com>
Co-authored-by: cyberhoward <cyberhoward@protonmail.com>
Updates the requirements on [env_logger](https://github.com/rust-cli/env_logger) to permit the latest version.
- [Release notes](https://github.com/rust-cli/env_logger/releases)
- [Changelog](https://github.com/rust-cli/env_logger/blob/main/CHANGELOG.md)
- [Commits](rust-cli/env_logger@v0.10.0...v0.11.0)

---
updated-dependencies:
- dependency-name: env_logger
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* show POC of generalized querier getter

* update

* fix merge and impl for all envs

* Doon't verify mockj error anymore

* Fixed test

* update `QueryHandler` and WasmQuerier functions

* fix smart query URL

* fix compile

* format and clippy

---------

Co-authored-by: Kayanski <kowalski.kowalskin@gmail.com>
* Added instantiate2

* some fixes

* Added instantiate2 addr getter

* Format

* Nits
* Added cw-multi-test 0.20.0

* Removed secp, removed ibc-relayer-types

* Removed ibc-relayer-types completely

* updated prost typesé

* Start testing authz

* Added authz test

* Added hd path index change inside sender and daemon

* Fixed sender options usageé

* add authz querier

* tx handler error with anyhow test

* format

* Added authz builder method for options

* Some nits

* Revert with_authz, added sender options setter

* Check rest of authz queries

* Added fee grant in daemon builder and interface

* Refactor daemon for simplified re-builderé

* Modifed async daemon as well

* Added docs for builder + nits

* Format

---------

Co-authored-by: Buckram <buckram123@gmail.com>
@Kayanski Kayanski changed the base branch from main to gh-pages February 5, 2024 10:59
@Kayanski Kayanski closed this Feb 5, 2024
@Kayanski
Copy link
Contributor Author

Kayanski commented Feb 5, 2024

This was closed by mistake because of a dumb change of branch to merge into.
The corresponding PR is here : #322

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.

7 participants