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.

Sign up for GitHub

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

Jump to bottom

refactor: add handler errror #160

Merged
merged 1 commit into from
Jan 5, 2023
Merged

refactor: add handler errror #160

merged 1 commit into from
Jan 5, 2023

Conversation

hoanguyenkh
Copy link
Collaborator

Description

Closes: #XXX

PR Checklist:

  • Have you read the CONTRIBUTING.md?
  • Does your PR follow the C4 patch requirements?
  • Have you rebased your work on top of the latest master?
  • Have you checked your code compiles? (make)
  • Have you included tests for any non-trivial functionality?
  • Have you checked your code passes the unit tests? (make test)
  • Have you checked your code formatting is correct? (go fmt)
  • Have you checked your basic code style is fine? (golangci-lint run)
  • If you added any dependencies, have you checked they do not contain any known vulnerabilities? (go list -json -m all | nancy sleuth)
  • If your changes affect the client infrastructure, have you run the integration test?
  • If your changes affect public APIs, does your PR follow the C4 evolution of public contracts?
  • If your code changes public APIs, have you incremented the crate version numbers and documented your changes in the CHANGELOG.md?

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

  • confirmed the correct type prefix in the PR title
  • confirmed all author checklist items have been addressed
  • confirmed that this PR does not change production code

@hoanguyenkh hoanguyenkh merged commit 6db8239 into astra Jan 5, 2023
@hoanguyenkh hoanguyenkh deleted the addErrHandler branch January 5, 2023 06:45
hoanguyenkh added a commit that referenced this pull request Feb 1, 2023
@hoanguyenkh @LampardNguyen234
feat: add module mint, feeburn (#157)
a957e5c 
* feat: add NewMinGasPriceDecorator

* feat: add module feeburn

* refactor fee burn

* update proto and add some test

* add integration test

* add test send batch transaction

* update integration test

* refactor add ante test

* add integration test feeburn

* dev/mint (#155)

* feat: add mint module's proto

* chore: update proto

* feat: add mint types

* chore: update mint types

* chore: update mint params

* test: add unittest for mint types

* test: update unittest for mint types

* feat: add mint keeper

* chore: remove inflation module

* feat: add API querier

* chore: update querier keys

* test: add unit tests for mint keeper

* test: update unit test for mint keeper

* feat: add mint module cli

* feat: add rest queries for mint module

* chore: update mint module cli

* feat: add mint module abci, simulation, genesis

* refactor: update mint simulation package

* test: add integration test for mint module

* docs(mint): add spec for the mint module

* chore: add third_party proto's

* chore: update main app

* chore: remove proto unused

* test: fix unittest invalid denom

* chore: remove code unused

* refactor: update astra app

* docs: add module feeburn

* fix: miss feeburn in ante handler

* refactor: reorder module

* chore(mint): add queryCirculatingSupply and queryBondedRatio methods

* test(mint): update unit tests

* feat: add feeburn rest api

* chore: update swagger

* chore: update script gen

* refactor: update proto

* test: add feeburn integration test

* chore: upadte init.sh

* feat: add get total_fee_burn in module fee

* chore: update init.sh

* test: update integration test feeburn

* test: add integration test evm tx

* test: update integration test

* test: upadte integration tests for mint

* test: update integration test cases

* fix: totalBurnAmount incorrect

* test: update integration tests for mint

* test: update test_fee_payer

* test: update test_fee_payer (#159)

* test: update integration tests

* test: update unittest for mint module

* refactor: add handler errror (#160)

* test(mint): disable integration tests

* fix: integer overflow (#161)

* fix: integer overflow

* refactor: change parse dec -> int

* refactor: add default DefaultMinGasPrice, DefaultMinGasMultiplier

* refactor: remove upgrade handler

* refactor: remove default MinGasPrice

* add fork code update voting period (#163)

* add fork code

* add fork code

* update UpgradeHeight

Co-authored-by: hoanguyenkh <[email protected]>

* update UpgradeHeight

* update UpgradeHeight

* change update name

* test: check fee burn when test fail

* chore: update changelog

* chore: update Genesis Distribution

* deps: bump ginkgo to v2.7.1, gomega to v1.26.0

* chore: update CHANGELOG.md and refactor go.mod

* refactor: update logic check burn amount

---------

Co-authored-by: hoanguyenkh <[email protected]>
Co-authored-by: Thanh Nguyen <[email protected]>
Co-authored-by: LampardNguyen234 <[email protected]>
Co-authored-by: hoanguyenkh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hoanguyenkh