Skip to content

Set custom handler comment to be changed v8 on 2026 Q4#123

Merged
TxCorpi0x merged 3 commits into
masterfrom
mehdi/custom-query-handler-2026-q4
Apr 10, 2026
Merged

Set custom handler comment to be changed v8 on 2026 Q4#123
TxCorpi0x merged 3 commits into
masterfrom
mehdi/custom-query-handler-2026-q4

Conversation

@TxCorpi0x

@TxCorpi0x TxCorpi0x commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Description

This pull request updates several deprecation comments throughout the codebase to clarify that the removal of legacy handlers and related functions is now planned for "v8" in 2026 Q4, instead of "v7". No functional or behavioral changes are introduced; only the planned removal version and date in TODO comments have been updated for clarity.

Deprecation comment updates:

  • Updated all relevant TODO comments in app/app.go, x/wasm/handler/message.go, and x/wasm/handler/query.go to indicate that legacy message and query handlers, as well as related functions, are now scheduled for removal in v8 (2026 Q4) instead of v7. [1] [2] [3] [4]

Reviewers checklist:

  • Try to write more meaningful comments with clear actions to be taken.
  • Nit-picking should be unblocking. Focus on core issues.

Authors checklist

  • Provide a concise and meaningful description
  • Review the code yourself first, before making the PR.
  • Annotate your PR in places that require explanation.
  • Think and try to split the PR to smaller PR if it is big.

This change is Reviewable

@TxCorpi0x TxCorpi0x requested a review from a team as a code owner April 9, 2026 13:14
@TxCorpi0x TxCorpi0x requested review from masihyeganeh, metalarm10, miladz68 and ysv and removed request for a team April 9, 2026 13:14

@metalarm10 metalarm10 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@metalarm10 reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on masihyeganeh, miladz68, and ysv).

@TxCorpi0x TxCorpi0x merged commit 6bbe931 into master Apr 10, 2026
13 of 14 checks passed
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.

2 participants