Skip to content

chore: Deprecate the set_node_account_id and node_account_id for Transaction#2476

Draft
manishdait wants to merge 2 commits into
hiero-ledger:mainfrom
manishdait:feat/deprecate-set-node_id
Draft

chore: Deprecate the set_node_account_id and node_account_id for Transaction#2476
manishdait wants to merge 2 commits into
hiero-ledger:mainfrom
manishdait:feat/deprecate-set-node_id

Conversation

@manishdait

Copy link
Copy Markdown
Contributor

Description:
This PR deprecate the set_node_account_id and node_account_id to use set_node_account_ids and aling with other sdk's

Related issue(s):

Fixes #2475

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Manish Dait <daitmanish88@gmail.com>
Signed-off-by: Manish Dait <daitmanish88@gmail.com>
@github-actions github-actions Bot added the approved Issue has been approved by team member label Jul 23, 2026
@manishdait manishdait changed the title feat: feat: Deprecate the set_node_account_id and node_account_id for Transaction Jul 23, 2026
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.96970% with 1 line in your changes missing coverage. Please review.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2476   +/-   ##
=======================================
  Coverage   95.27%   95.28%           
=======================================
  Files         164      164           
  Lines       10485    10494    +9     
=======================================
+ Hits         9990     9999    +9     
  Misses        495      495           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@manishdait manishdait changed the title feat: Deprecate the set_node_account_id and node_account_id for Transaction chore: Deprecate the set_node_account_id and node_account_id for Transaction Jul 23, 2026
@aceppaluni aceppaluni removed the approved Issue has been approved by team member label Jul 24, 2026
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.

Align node account ID APIs with other SDKs

2 participants