Master branch update to support Columbus-4 #422
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[Feature] proposal events
[Bug] slashing codec
[Bugfix] Fill address to signing info
* append new msgs - assoicate oracle prevote & vote with updated whitelist param structure to include illiquid factor on denom property * append querier for associate oracle prevote, cmd & rest interface to send associate oracle prevote & vote * append simulation code for associate msgs * implement import & export for associate prevote * delete associate prevotes at export for zero height * change associate to aggregate * update swagger docs * store aggregate vote in the keeper to reduce file I/O operation * add genesis export import feature for aggregate vote * add codec & cmd tx * introduce vote targets to avoid directly refering params.Whitelist from oracle slashing, change sdk.DecCoins to types.ExchangeRateTuples on types.AggregateExchangeRateVote * apply default value for default VotesTarget at genesis import step * store illiquid factors for safe swap & make vote hash type * bugfix - delete aggregate votes&prevotes after tally * ignore vote from the voter who did aggregate vote * simulation code fix * skip reward allocation when the reward is zero * add max exchange rates length limit on aggregate vote msg & return deatils err for aggregate prevote/vote * for internal map structure, directly stringify validator address to prevent unnecessary bech32fy works * change to use unique tobin taxes per denom in oracle params & variable name changes * remove unused yaml definition * fix lint
* bump sdk version to v0.38.3 * localnet script update * add clean command & update makefile * fix docker image name * prevent bank simulation from returning err on insufficient fee error
Co-authored-by: Paul Kim <[email protected]>
* localnet script update * add clean command & update makefile * prevent bank simulation from returning err on insufficient fee error * add burn address * change burn place from handler to endbloker
* localnet script update * add clean command & update makefile * add wasm module * decrease wasm test weight * charge tax on the msgs from contract & add treasury querier for tax-rate and tax-cap to enable tax estimation on the contract * allow reference msg * add tax charging test * add client code to auto compute tax on instantiate & execute contract * not execute deduct fee when computed taxes are zero * change hex bytes to base64 for wasm module * remove estimated gas consume test Co-authored-by: Paul Kim <[email protected]>
* add MsgSwapSend feature * append MsgSwapSend with tax deduct & wasm interface * change receiver to toAddress * convention for parameters to be kebab-cased Co-authored-by: Paul Kim <[email protected]>
…350) * add wasm params querier * add querier test * fix rest api to load from address * align wasm query msg to json string * remove uncessary string wrapper on rest interface * align rest endpoint expression from singular to plural * fix swagger example * add json validation for wasm endpoints
* Bump cosmwasm v0.9.x & append migration feature * add testcase and client code for migrate & update-owner * add simulation code for updates * delete data * Bump cosmwasm v0.9.2 & go-cosmwasm v0.9.1 * fix swagger example Co-authored-by: Paul Kim <[email protected]>
* copy from msgauth module from https://github.com/regen-network/cosmos-sdk/tree/regen-network/msg_authorization * finising msgauth interfaces * swagger update & client interface update * remove todo * convert rest grant limit to coins * add msgauth simulation * fix simulation * fix client description * remove redundent error tag * swagger: MsgAuth tag to Authorization Co-authored-by: Paul Kim <[email protected]>
* changelog update
* add columbus-4 genesis migration script * implement indicators querier for tax rate update * add public option at starting rest-server & enforce to use tx.height option when public option is on * fix(treasury): indicator endpoint feat(treasury): indicator swagger definition * fix: stringify yes and no map keys (#379) * change module account name & permission key * apply core module migration script * add pretty optiong * add new modules: msgauth, wasm, mint to script * update script for wasm params and expected time * remove wasm gas_multiplier params from genesis migration script * add changelog Co-authored-by: Paul Kim <[email protected]> Co-authored-by: Jesse Chung <[email protected]>
…ed ABCI errors that all map to undefined internal errors. (#414) This PR maps to the InvalidCoinsError and makes for an easier time debugging AggregateVoteMessages
Merge master to resolve conflict develop<>master
* update wasm event system to restrict from_contract logging to whitelist * save wasm.toml right after new contract added to whitelist Co-authored-by: Paul Kim <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #422 +/- ##
==========================================
- Coverage 72.87% 67.52% -5.36%
==========================================
Files 73 117 +44
Lines 3089 5071 +1982
==========================================
+ Hits 2251 3424 +1173
- Misses 731 1433 +702
- Partials 107 214 +107
|
This reverts commit 9695e97.
* cosmwasm cache update * bump cosmwasm to v0.10.3 Co-authored-by: Paul Kim <[email protected]>
hanjukim
previously approved these changes
Nov 11, 2020
* recover from out-of-gas panic * fix testing for query panic * format recovered error depends on the error type
* add softfork condition to staking wasm querier * formatting * update error codes bigger than 1 * handle prev error to internal error * handle prev error to internal error * fix wrong error type * add wasm execute msg size limit update * remove unstable staking begin blocker
hanjukim
approved these changes
Nov 23, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of changes
Merge v0.4.1 updates to Master branch
Report of required housekeeping
(FOR ADMIN) Before merging