Skip to content

Multi transaction and expected public key - #41

Open
krigga wants to merge 37 commits into
LedgerHQ:developfrom
krigga:multi-tx
Open

Multi transaction and expected public key#41
krigga wants to merge 37 commits into
LedgerHQ:developfrom
krigga:multi-tx

Conversation

@krigga

@krigga krigga commented Jan 29, 2026

Copy link
Copy Markdown

Checklist

  • App update process has been followed
  • Target branch is develop
  • Application version has been bumped

This PR includes two new features:

  • Multi-transaction - the ability to include more than 1 transaction (in TON's terms, more than 1 message) in a single signature to be executed together
  • Expected public key - clients may send an "expected public key" to Ledger together with transaction data, and Ledger will refuse to sign if the key does not match

@codecov-commenter

codecov-commenter commented Feb 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.05%. Comparing base (e5c651f) to head (fa52f64).
⚠️ Report is 11 commits behind head on develop.

Files with missing lines Patch % Lines
src/common/format_bigint.c 66.66% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop      #41       +/-   ##
============================================
+ Coverage    68.42%   91.05%   +22.63%     
============================================
  Files            8       20       +12     
  Lines          247      995      +748     
  Branches         0      116      +116     
============================================
+ Hits           169      906      +737     
- Misses          78       85        +7     
- Partials         0        4        +4     
Flag Coverage Δ
unittests 91.05% <71.42%> (+22.63%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tdejoigny-ledger

Copy link
Copy Markdown

@krigga could you please fix the fails in the CI ?
and bump the app version

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.

4 participants