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

Add tax2gas module and move all handle fees logic to tax2gas module #491

Closed
wants to merge 51 commits into from
Closed

Add tax2gas module and move all handle fees logic to tax2gas module #491

wants to merge 51 commits into from

Conversation

phamminh0811
Copy link
Collaborator

Summary of changes

Close #482

Report of required housekeeping

  • Github issue OR spec proposal link
  • Wrote tests
  • Updated API documentation (client/lcd/swagger-ui/swagger.yaml)
  • Added a relevant changelog entry: clog add [section] [stanza] [message]

(FOR ADMIN) Before merging

  • Added appropriate labels to PR
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
  • Confirm added tests are consistent with the intended behavior of changes
  • Ensure all tests pass

phamminh0811 and others added 27 commits June 24, 2024 11:18
@phamminh0811
lint
7500c39
@phamminh0811
proto lint
1599911
@phamminh0811
revert change in protocgen
8fe545d
@expertdicer
add tax2gas module and implementation
c40fa96
@expertdicer
remove redundant files
22a7186
@expertdicer
remove wasm file
ef4264f
@expertdicer
remove wasm file
226de57
@expertdicer
change proto
a2ae5c9
@expertdicer
v8_1 upgrade handler
4cd85c9
@expertdicer
rename var and add comment
cce285f
@expertdicer
remove redundant check
af547c2
@expertdicer
add comment and rename var
1203f0f
@expertdicer
correctly set ctx.Value
b272150
@fragwuerdig @expertdicer
feat: oracle split as per governance (#495)
2db4512 
Co-authored-by: expertdicer <[email protected]>
@expertdicer
remove tax2gas subspace and add authority
c131733
@expertdicer
register msg service
bb99053
@expertdicer
add msg update params
92ef82c
@expertdicer
register codec and unit tests
c016cc6
@expertdicer
proto update
e6d8242
@expertdicer
refactor tax2gas querier
023cdfe
@fragwuerdig
fix: wrong interface cast
829a7d2
@StrathCole @fragwuerdig
Adjust oracle split logic (#496)
01932fe 
Co-authored-by: Till Ziegler <[email protected]>
@expertdicer
refactor structure
2fc2c60
@expertdicer
adding unit tests
2d94166
@expertdicer
deduct from tx origin
a16d4df
@phamminh0811
Allow to paid by multiple fees denom
ed0ad82
@expertdicer
consume tax gas in wasm plugin
d57e4ce
phamminh0811 and others added 20 commits July 17, 2024 12:25
@phamminh0811
Add test scenario to tax2gas
9131a18
@expertdicer
Merge branch 'minh/tax2gas' of github-expertdicer:classic-terra/core …
f615e83 
…into minh/tax2gas
@expertdicer
add simulate condition
d83b8b3
@expertdicer
update tests and checkTxFee
052f7fb
@phamminh0811
lint
e2b6dc1
@phamminh0811
lint
2f779bf
@phamminh0811
fix interchain test
8ecad7e
@expertdicer
update tests and minor fix
b7aa915
@expertdicer
Merge branch 'minh/tax2gas' of github-expertdicer:classic-terra/core …
d66c1f4 
…into minh/tax2gas
@phamminh0811 @expertdicer
fix: use wasmvm v1.5.2 to prevent memory leak (#497)
51a080f 
Co-authored-by: expertdicer <[email protected]>
@phamminh0811
Move burn tax process to post handler
3e816ea
@expertdicer
add bypass msg types and max gas, unit tests
af2336c
@expertdicer
Merge branch 'minh/tax2gas' of github-expertdicer:classic-terra/core …
3f22cf0 
…into minh/tax2gas
@expertdicer
update tests
75bd097
@expertdicer
Merge branch 'main' into minh/tax2gas
693a641
@expertdicer
update post handler
fb0344f
@phamminh0811
Use taxGasMeter in sdk to handle tax2Gas logic (#499)
3deac13
@phamminh0811
fix e2e test
56ea70b
@expertdicer
fix test
1327e9d
@expertdicer
Merge branch 'minh/tax2gas' of github-expertdicer:classic-terra/core …
644d5f4 
…into minh/tax2gas
@inon-man inon-man deleted the branch feat/tax2gas July 25, 2024 08:16
@inon-man inon-man closed this Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@expertdicer expertdicer Awaiting requested review from expertdicer

@DongLieu DongLieu Awaiting requested review from DongLieu

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@phamminh0811 @inon-man @expertdicer @fragwuerdig @StrathCole