- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Dec 6 2024
Dec 5 2024
add menu glow value
nice, Cashtab has needed this. Much more maintainable.
In D17294#392907, @PiRK wrote:I will build a beta AppImage and test that the release actually works with this bump. But I really see no reason why it wouldn"t. Trezorlib is pure python so no weird compatibility surprises to be expected related to compiled libs. And it supports all python versions >=8 (we package with 3.11
I will build a beta AppImage and test that the release actually works with this bump. But I really see no reason why it wouldn"t. Trezorlib is pure python so no weird compatibility surprises to be expected related to compiled libs. And it supports all python versions >=8 (we package with 3.11
At some point we can also change the regular (non-deterministic) requirements.txt to force >= 0.13.9 and drop the compat.py code. It can be done in a separate diff, though. I'm still trying to think of potential downsides of doing so. Electrum maintains support for trezor>=0.13.0 mainly because there are packagers on various Linux distros who have to work with older trezor packages. We don't have this issue with Electrum ABC.
put background image back
remove unused import
It works with only a minor change in Electrum ABC code
Fix comment
Sanitize the imports
I still think it's not the best option but I'm not dying on the hill since both are valid anyway.
fix CheckNegate
add rationales
typo patch
Rebase, feedback
Dec 4 2024
build lint
improve file organization as suggested
address review, remove CScriptNum10 and replace tests with tests that test the behavior of CScriptNum instead of testing whether it matches CScriptNum10
address review
An optional codec for the server has been upstreamed (https://github.com/karyontech/karyon/pull/10) and we can leverage this to add the weird \n that fulcrum expects.
This makes it compatible with electrum with no parsing change required (but we still need to send the jsonrpc field which is required by the standard).