Skip to content

Testnet Missed upgrade steps#102

Open
TxCorpi0x wants to merge 3 commits intorelease-v6from
mehdi/testnet-v6-upgrade-patch
Open

Testnet Missed upgrade steps#102
TxCorpi0x wants to merge 3 commits intorelease-v6from
mehdi/testnet-v6-upgrade-patch

Conversation

@TxCorpi0x
Copy link
Contributor

@TxCorpi0x TxCorpi0x commented Mar 10, 2026

Description

This pull request introduces a patch upgrade for the v6 release, focusing on updating the PSE module parameters to testnet defaults and clarifying which migrations have already been performed in previous upgrades. The upgrade handler is renamed and unnecessary migration steps are commented out to prevent redundancy.

Upgrade handler changes:

  • Renamed the upgrade handler from "v6" to "v6-params-patch" to reflect its purpose as a patch.
  • Updated the StoreUpgrades and migration steps to comment out actions that were already completed in the previous v6.0.0 upgrade, preventing duplicate execution and clarifying upgrade history. [1] [2]

PSE module Changes:

  • Added the TestnetV6Patch function in pse_init.go to set PSE module parameters to testnet defaults, and integrated this step into the upgrade handler. [1] [2]

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 marked this pull request as ready for review March 10, 2026 07:05
@TxCorpi0x TxCorpi0x requested a review from a team as a code owner March 10, 2026 07:05
@TxCorpi0x TxCorpi0x requested review from masihyeganeh, metalarm10, miladz68 and ysv and removed request for a team March 10, 2026 07:05
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.

1 participant