Skip to content

feat: simple fees#1213

Open
ivaylonikolov7 wants to merge 4 commits intomainfrom
feat/simple-fees
Open

feat: simple fees#1213
ivaylonikolov7 wants to merge 4 commits intomainfrom
feat/simple-fees

Conversation

@ivaylonikolov7
Copy link
Copy Markdown
Contributor

@ivaylonikolov7 ivaylonikolov7 commented Feb 8, 2026 •
edited
Loading

Description:

  • Add chunked transaction fixes, per-chunk transaction IDs, and FeeEstimateQuery support.
  • Fix chunked transaction list to build one transaction per (chunk, node) instead of one per node
  • Add chunk_interval_nanos to ChunkData and use distinct transaction IDs per chunk when set (FileAppend and TopicMessageSubmit)
  • Add TransactionId::with_valid_start_offset for per-chunk IDs
  • Add unit tests for chunked FileAppend and TopicMessageSubmit (transaction count and distinct IDs per chunk)
  • Add FeeEstimateQuery and mirror-node fee estimate API integration
  • Add e2e tests for fee estimate query

** Fixes ** - #1147

Checklist

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

@ivaylonikolov7 ivaylonikolov7 requested review from a team as code owners February 8, 2026 23:42
ivaylonikolov7 and others added 3 commits February 9, 2026 01:43
@ivaylonikolov7 @cursoragent
fix(chunked): one transaction per (chunk, node) and distinct transact…
8519db5 
…ion ID per chunk

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Ivaylo Nikolov <ivaylo.nikolov@limechain.tech>
@ivaylonikolov7 @cursoragent
test(transaction): add unit tests for chunked FileAppend and TopicMes…
90f908e 
…sageSubmit

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Ivaylo Nikolov <ivaylo.nikolov@limechain.tech>
@ivaylonikolov7 @cursoragent
feat: add FeeEstimateQuery and e2e tests
3af5ee7 
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Ivaylo Nikolov <ivaylo.nikolov@limechain.tech>
@ivaylonikolov7
fix: update solo action
64acca9 
Signed-off-by: Ivaylo Nikolov <ivaylo.nikolov@limechain.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nathanklick nathanklick Awaiting requested review from nathanklick nathanklick is a code owner automatically assigned from hiero-ledger/github-maintainers

@Sheng-Long Sheng-Long Awaiting requested review from Sheng-Long Sheng-Long is a code owner automatically assigned from hiero-ledger/hiero-sdk-rust-maintainers

@RickyLB RickyLB Awaiting requested review from RickyLB RickyLB is a code owner automatically assigned from hiero-ledger/hiero-sdk-rust-committers

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ivaylonikolov7
⚡