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.

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

RPC: Include optional linked_account field in account_history, block_info, and blocks_info RPCs #4841

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

Exxenoz
Copy link
Contributor

@Exxenoz Exxenoz commented Feb 9, 2025

This PR adds an optional parameter include_linked_account to the account_history, block_info, and blocks_info RPC endpoints.

When enabled, the response will include a new field linked_account, which returns the linked account associated with the block (or "0" if no linked account exists/was found e.g. due to pruning). The related tests have been updated accordingly.

Issues resolved
This PR is an extension to PR #4833

Exxenoz and others added 3 commits February 9, 2025 17:30
- Add proper `linked_account` to the block content for send, receive, and open blocks when the `include_linked_account` option is enabled.
- For change and epoch blocks, `linked_account` is set to "0", maintaining consistency with the `source` field in the blocks_info RPC response.

Co-authored-by: clemahieu <[email protected]>
Co-authored-by: gr0vity <[email protected]>
@gr0vity-dev-bot
Copy link

gr0vity-dev-bot commented Feb 9, 2025

Test Results for Commit 34e0394

Pull Request 4841: Results
Overall Status:

Test Case Results

  • 5n4pr_conf_10k_bintree: PASS (Duration: 114s)
  • 5n4pr_conf_10k_change: PASS (Duration: 177s)
  • 5n4pr_conf_change_dependant: PASS (Duration: 133s)
  • 5n4pr_conf_change_independant: PASS (Duration: 127s)
  • 5n4pr_conf_send_dependant: PASS (Duration: 117s)
  • 5n4pr_conf_send_independant: PASS (Duration: 116s)
  • 5n4pr_rocks_10k_bintree: PASS (Duration: 114s)
  • 5n4pr_rocks_10k_change: PASS (Duration: 153s)

Last updated: 2025-02-09 17:26:43 UTC

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