Skip to content

improvement(RPCs): group staking rpcs under a namespace#2372

Merged
shamardy merged 5 commits intodevfrom
staking-rpcs
Feb 27, 2025
Merged

improvement(RPCs): group staking rpcs under a namespace#2372
shamardy merged 5 commits intodevfrom
staking-rpcs

Conversation

@onur-ozkan
Copy link
Copy Markdown

@onur-ozkan onur-ozkan commented Feb 26, 2025

Breaking Changes

This PR re-organizes staking-related RPC methods under a single namespace called staking, and marks them as experimental. The changes made aim to improve consistency and future extensibility within the staking module.

Renamed RPC Methods

The following RPC methods have been renamed to align with a more structured namespace:

  • add_delegationexperimental::staking::delegate
  • remove_delegationexperimental::staking::undelegate
  • get_staking_infosexperimental::staking::query::delegations

Payload Changes

The request payload for get_staking_infos (now experimental::staking::query::delegations) has been updated.

Previous Format:

{
    "userpass": "pass",
    "mmrpc": "2.0",
    "method": "get_staking_infos",
    "params": {
        "coin": "tQTUM"
    },
    "id": 0
}

Updated Format:

{
    "userpass": "pass",
    "mmrpc": "2.0",
    "method": "experimental::staking::query::delegations",
    "params": {
        "coin": "tQTUM",
        "info_details": {
            "type": "Qtum"
        }
    },
    "id": 0
}

Note: Tendermint staking related breaking changes are are not listed here, as we never documented them anywhere yet.

Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Copy link
Copy Markdown
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

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

LGTM!

@shamardy
Copy link
Copy Markdown
Collaborator

@onur-ozkan please open docs issue

@onur-ozkan
Copy link
Copy Markdown
Author

@onur-ozkan please open docs issue

It's done already: GLEECBTC/komodo-docs-mdx#420

@shamardy shamardy merged commit dce9da8 into dev Feb 27, 2025
30 checks passed
@onur-ozkan onur-ozkan deleted the staking-rpcs branch February 27, 2025 16:12
dimxy pushed a commit that referenced this pull request Mar 5, 2025
* dev:
  feat(rpc): add is_success field to legacy MySwapStatusResponse (#2371)
  fix(key-derivation): use stored Argon2 parameters instead of default values (#2360)
  fix(tests): stabilize `tendermint_coin::test_claim_staking_rewards` (#2373)
  improvement(RPCs): group staking rpcs under a namespace (#2372)
  feat(tendermint): claim delegation rewards (#2351)
  fix(eth-tpu): remove state from funding validation (#2334)
  improvement(rpc-server): rpc server dynamic port allocation (#2342)
  fix(tests): fix or ignore unstable tests (#2365)
  fix(fs): make `filter_files_by_extension` return only files (#2364)
  fix(derive_key_from_path): check length of current_key_material (#2356)
  chore(release): bump mm2 version to 2.4.0-beta (#2346)
  fix(tests): add additional testnet sepolia nodes to test code (#2358)
  fix(swaps): maintain legacy compatibility for negotiation messages (#2353)
  refactor(SwapOps): impl defaults for protocol specific swapops fns (#2354)
  feat(tpu-v2): provide swap protocol versioning (#2324)
  feat(wallet): add change mnemonic password rpc (#2317)
dimxy pushed a commit that referenced this pull request Mar 21, 2025
* dev:
  fix(tx-history): fix unhandled IBC and HTLC events (#2385)
  improvement(dep-stack): replace deprecated `instant` dependency  (#2391)
  feat(tendermint): staking queries (#2377)
  refactor(eth): use trait addr_to_string method instead of old function (#2348)
  fix(ci): use correct rustup component syntax in fmt-and-lint job (#2390)
  refactor(tx-query): use TxSearchRequest for tx queries (#2384)
  refactor(tpu-v2): allow to skip p2p message with taker payment spend preimage for eth (#2359)
  feat(rpc): add is_success field to legacy MySwapStatusResponse (#2371)
  fix(key-derivation): use stored Argon2 parameters instead of default values (#2360)
  fix(tests): stabilize `tendermint_coin::test_claim_staking_rewards` (#2373)
  improvement(RPCs): group staking rpcs under a namespace (#2372)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants