Skip to content

fix: ibc handling in wasm#653

Merged
kien6034 merged 6 commits intoclassic-terra:mainfrom
orbitorg:main
Apr 27, 2026
Merged

fix: ibc handling in wasm#653
kien6034 merged 6 commits intoclassic-terra:mainfrom
orbitorg:main

Conversation

@TropicalDog17
Copy link
Copy Markdown
Contributor

  • remove fork code to use upstream
  • remove unused code
  • Problem: cometbft not up to date

  • add v14_2

  • bump version

  • bump version

  • bump version

  • fix legacy query staking (Build upgrade testing for wasm #79)

  • fix legacy query staking

  • bet

  • fix legacy params for validator delegation fallback

  • Bet

  • bet


  • consistent alias

Summary of changes

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

hoank101 and others added 2 commits April 25, 2026 22:44
* - fix ibc handling in wasm (#77)

- remove fork code to use upstream
- remove unused code

* Problem: cometbft not up to date

* add v14_2

* bump version

* bump version

* bump version

* fix legacy query staking (#79)

* fix legacy query staking

* bet

* fix legacy params for validator delegation fallback

* Bet

* bet

---------

Co-authored-by: Khanh Hoa <hoa@notional.ventures>
Co-authored-by: Tuan Tran <tropicaldog17@gmail.com>

* consistent alias

---------

Co-authored-by: StrathCole <7449529+StrathCole@users.noreply.github.com>
Co-authored-by: Kien Trinh <51135161+kien6034@users.noreply.github.com>
Co-authored-by: Tuan Tran <tropicaldog17@gmail.com>
@StrathCole StrathCole requested review from Copilot and fragwuerdig and removed request for Copilot April 26, 2026 05:50
@kien6034 kien6034 merged commit 9a5ee56 into classic-terra:main Apr 27, 2026
31 checks passed
Comment thread app/keepers/keepers.go
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

		appKeepers.TaxExemptionKeeper,
		appKeepers.TreasuryKeeper,
		appKeepers.AccountKeeper,
		appKeepers.TaxKeeper,

Why this is removed ? ( just a question)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was not used in that module.

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.

6 participants