-
Notifications
You must be signed in to change notification settings - Fork 6
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
XATP module #73
Merged
Merged
XATP module #73
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
JoowonYun
force-pushed
the
feature/xatp-economy
branch
from
March 10, 2023 06:58
d7418f9
to
6553f5d
Compare
JoowonYun
force-pushed
the
feature/xatp-economy
branch
from
March 13, 2023 08:52
43f1412
to
79b55c5
Compare
JoowonYun
force-pushed
the
feature/xatp-economy
branch
from
March 14, 2023 08:16
79b55c5
to
fd803ab
Compare
JoowonYun
force-pushed
the
feature/xatp-economy
branch
2 times, most recently
from
March 20, 2023 07:36
c3053ba
to
5e3bbbe
Compare
JoowonYun
force-pushed
the
feature/xatp-economy
branch
from
March 20, 2023 07:51
5e3bbbe
to
743d801
Compare
JoowonYun
force-pushed
the
feature/xatp-economy
branch
from
March 21, 2023 09:35
743d801
to
ba0cb66
Compare
* 4-nodes automated integration test infrastructure (#46) * feat: added working Dockerfile * test: multinode docker & docker-compose * misc: added integration test in github action * misc: divided unit test & integration test * fix: added ETH port * test: added basis lib * misc: added test trigger * test: added a test - delegation * test: added test trigger * test: added tests: store code * test: added test - InstantiateContract * test: added test - contract execution * docs: added README of integration test * misc: added go setup before integration test * test: automated all prepared genesis.json creation process resolves #48 * test: changed broadcast mode as block and delete sleep * test: updated common testing part * test: added xatp tests * test: fixed - get simulation error from sync mode
yoosah
approved these changes
Mar 30, 2023
caramis
approved these changes
Mar 30, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
JoowonYun
added a commit
that referenced
this pull request
Feb 1, 2024
* add xatp add upgrade * fix: correct proto path * fix: correct module name * fix: mempool fee decorator test * refactor: remove unused code * feat: changed to manage XATP with map * feat: add to xatp query * refactor: simple conditions and calculations * feat: manage XATP as a proposal * refactor: remove unused SmartQueryGasLimit * refactor: ante MempoolFeeDecorator * refactor: ante DeductFeeDecorator * fix: exception handling when payer is empty in xatp module * fix: fee precision * feat: change payer to module account * feat: apply tax-rate * feat: xatp economy * feat: fund xatp pool & query pool * fix: typo * fix: validate simulation * refactor: merge upgrade (xatp & align gas price) * feat: gas refund when mempool check * refactor: increased precision in simulate * fix: change XATP from dec to int * fix: change the xatp param rate to be greater than 1 * fix: calculate reward pool * fix: check fee when simulate * Added integartion tests for XATP module (#60) * 4-nodes automated integration test infrastructure (#46) * feat: added working Dockerfile * test: multinode docker & docker-compose * misc: added integration test in github action * misc: divided unit test & integration test * fix: added ETH port * test: added basis lib * misc: added test trigger * test: added a test - delegation * test: added test trigger * test: added tests: store code * test: added test - InstantiateContract * test: added test - contract execution * docs: added README of integration test * misc: added go setup before integration test * test: automated all prepared genesis.json creation process resolves #48 * test: changed broadcast mode as block and delete sleep * test: updated common testing part * test: added xatp tests * test: fixed - get simulation error from sync mode * refactor: variable names --------- Co-authored-by: 임진승 <[email protected]> Co-authored-by: psy2848048 <[email protected]>
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.
Effects
Major Reviewer
@green-blue0 @yoosah
Background
Summary
Checklist