Skip to content

Release 26.03.0#840

Merged
odudex merged 66 commits intomainfrom
release-26.03.0
Mar 26, 2026
Merged

Release 26.03.0#840
odudex merged 66 commits intomainfrom
release-26.03.0

Conversation

@odudex
Copy link
Copy Markdown
Member

@odudex odudex commented Mar 24, 2026

What is this PR for?

26.03.0 Release

odudex and others added 30 commits October 27, 2025 08:16
add missing minor changes from previous changelog
adjust BULL coordinator labels and links
* add new device: Embed Fire

custom display initialization for Embed Fire

* add touchscreen tests
not only for Embed Fire

* Embed Fire added to simulator and docs (#53)

---------

Co-authored-by: Tads <tadeubas@gmail.com>
* Update Maixpy to return bytes object from binary QR codes

* Update Datum to auto convert, when possible, binary QRs to strings

* Ensure addresses become strings when scanned

* When encryption-key is scanned, try to decode as str

---------

Co-authored-by: Jean Do <117163651+jdlcdl@users.noreply.github.com>
add WonderK to release script
odudex and others added 20 commits December 9, 2025 10:04
*feat: loading a descriptor sets wallet attributes

*feat: remove the feature to load xpubs as they were descriptors, with or without assumptions.
* Simplify internal key validation for taproot

There is no standard for how to generate the taproot keypath chaincode to be used with the NUMS pubkey.
Liana does it by hashing the taptree keys;
Coldcard uses a random chaincode;
Nunchuck hashes the taptree keys, but remove duplicates and sort the keys before hashing.
This fix changes the verification logic to check only the NUMS pubkey and ignore the chaincode.

* Allow loading of non-deterministic chain code in wallet

The wallet now loads the non-deterministic chain code without raising an error, indicating it is considered valid.
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

❌ Patch coverage is 97.01987% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.31%. Comparing base (8f0fd5e) to head (941c4df).
⚠️ Report is 68 commits behind head on main.

Files with missing lines Patch % Lines
src/krux/wallet.py 93.61% 3 Missing ⚠️
src/krux/pages/home_pages/wallet_descriptor.py 97.10% 2 Missing ⚠️
src/krux/camera.py 75.00% 1 Missing ⚠️
src/krux/display.py 50.00% 1 Missing ⚠️
src/krux/key.py 85.71% 1 Missing ⚠️
src/krux/pages/datum_tool.py 96.15% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #840      +/-   ##
==========================================
+ Coverage   97.10%   97.31%   +0.21%     
==========================================
  Files          82       83       +1     
  Lines       10422    10573     +151     
==========================================
+ Hits        10120    10289     +169     
+ Misses        302      284      -18     

☔ View full report in Codecov by Sentry.
📢 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.

@tadeubas
Copy link
Copy Markdown
Member

I wouldn’t add commit eee8e95 . It doesn’t provide any real benefit, only invalidates the build cache for everyone. This isn’t really an issue anyway, since once the command succeeds, it’s cached.

@odudex
Copy link
Copy Markdown
Member Author

odudex commented Mar 24, 2026

I wouldn’t add commit eee8e95 . It doesn’t provide any real benefit, only invalidates the build cache for everyone. This isn’t really an issue anyway, since once the command succeeds, it’s cached.

Just testing here. Build script will demand more updates.

@tadeubas
Copy link
Copy Markdown
Member

If it requires further changes, wouldn’t it be better to open a separate PR for that?

@tadeubas
Copy link
Copy Markdown
Member

ACK, reviewed the code changes. I can’t mark it as approved since I don’t have the required permissions.

@odudex odudex merged commit 76feb7b into main Mar 26, 2026
16 checks passed
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.

7 participants