Page MenuHomePhabricator
Feed All Stories

Dec 6 2024

hazzarust retitled D17292: [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer from [BITCOIN-ABC] Allowing rust analyser to work in web/explorer to [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer.
Dec 6 2024, 01:26
hazzarust retitled D17292: [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer from [BITCOIN-ABC] Changes to rust analyser to [BITCOIN-ABC] Allowing rust analyser to work in web/explorer.
Dec 6 2024, 01:26
hazzarust updated the summary of D17292: [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer.
Dec 6 2024, 01:23
hazzarust updated the summary of D17292: [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer.
Dec 6 2024, 01:22
hazzarust retitled D17292: [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer from Changes to rust analyzer to [BITCOIN-ABC] Changes to rust analyser.
Dec 6 2024, 01:21
hazzarust updated the summary of D17295: [BITCOIN-ABC CI] Added `cargo build --locked` CI to web/explorer && `explorer.Dockerfile`.
Dec 6 2024, 01:21
hazzarust updated the summary of D17292: [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer.
Dec 6 2024, 01:20
hazzarust published D17292: [BITCOIN-ABC][WEB] Allowing rust analyser to work in web/explorer for review.
Dec 6 2024, 01:20
hazzarust published D17295: [BITCOIN-ABC CI] Added `cargo build --locked` CI to web/explorer && `explorer.Dockerfile` for review.
Dec 6 2024, 01:19

Dec 5 2024

tobias_ruck accepted D17185: [chronik] Create a new optional JSON RPC interface.
Dec 5 2024, 23:59
tobias_ruck closed D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.
Dec 5 2024, 23:56
tobias_ruck committed rABCd9bedb004bfb: [64-bit ints] Change `CScriptNum::getint` to return `int64_t` (authored by tobias_ruck).
[64-bit ints] Change `CScriptNum::getint` to return `int64_t`
Dec 5 2024, 23:56
tobias_ruck closed D17290: [64-bit ints] Add script_tests for `getint` coverage.
Dec 5 2024, 23:48
tobias_ruck committed rABCf67fd8dd5fd8: [64-bit ints] Add script_tests for `getint` coverage (authored by tobias_ruck).
[64-bit ints] Add script_tests for `getint` coverage
Dec 5 2024, 23:48
PiRK updated the summary of D17299: [electrum] fix the minimum python version.
Dec 5 2024, 22:50
PiRK requested review of D17299: [electrum] fix the minimum python version.
Dec 5 2024, 22:49
johnkuney updated the diff for D17296: [cashtab] Cleanup theme file.

add menu glow value

Dec 5 2024, 22:44
PiRK requested review of D17298: [electrum] bump libudev-dev for the AppImage release.
Dec 5 2024, 22:43
bytesofman closed D17297: Revert "[Cashtab] Convert wallet context to ts".
Dec 5 2024, 22:42
bytesofman committed rABCb88a433009ea: Revert "[Cashtab] Convert wallet context to ts" (authored by bytesofman).
Revert "[Cashtab] Convert wallet context to ts"
Dec 5 2024, 22:41
bytesofman added a reverting change for rABCd42543241000: [Cashtab] Convert wallet context to ts: rABCb88a433009ea: Revert "[Cashtab] Convert wallet context to ts".
Dec 5 2024, 22:41
bytesofman added a reverting change for D17288: [Cashtab] Convert wallet context to ts: rABCb88a433009ea: Revert "[Cashtab] Convert wallet context to ts".
Dec 5 2024, 22:41
johnkuney added inline comments to D17296: [cashtab] Cleanup theme file.
Dec 5 2024, 22:39
johnkuney accepted D17297: Revert "[Cashtab] Convert wallet context to ts".
Dec 5 2024, 22:31
bytesofman published D17297: Revert "[Cashtab] Convert wallet context to ts" for review.
Dec 5 2024, 22:28
bytesofman added a reverting change for D17288: [Cashtab] Convert wallet context to ts: D17297: Revert "[Cashtab] Convert wallet context to ts".
Dec 5 2024, 22:26
bytesofman added a reverting change for rABCd42543241000: [Cashtab] Convert wallet context to ts: D17297: Revert "[Cashtab] Convert wallet context to ts".
Dec 5 2024, 22:26
bytesofman requested changes to D17296: [cashtab] Cleanup theme file.

nice, Cashtab has needed this. Much more maintainable.

Dec 5 2024, 22:22
Fabien closed D17294: [electrum] Better support for Trezor safe devices.
Dec 5 2024, 22:16
Fabien committed rABCd275d0e46129: [electrum] Better support for Trezor safe devices (authored by Fabien).
[electrum] Better support for Trezor safe devices
Dec 5 2024, 22:16
Fabien added a comment to D17294: [electrum] Better support for Trezor safe devices.
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

Dec 5 2024, 22:16
bytesofman closed D17288: [Cashtab] Convert wallet context to ts.
Dec 5 2024, 22:15
bytesofman committed rABCd42543241000: [Cashtab] Convert wallet context to ts (authored by bytesofman).
[Cashtab] Convert wallet context to ts
Dec 5 2024, 22:15
PiRK added a comment to D17294: [electrum] Better support for Trezor safe devices.

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

Dec 5 2024, 22:04
PiRK accepted D17294: [electrum] Better support for Trezor safe devices.

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.

Dec 5 2024, 21:53
johnkuney updated the diff for D17296: [cashtab] Cleanup theme file.

put background image back

Dec 5 2024, 21:23
johnkuney updated the diff for D17296: [cashtab] Cleanup theme file.

remove unused import

Dec 5 2024, 21:09
emack accepted D17288: [Cashtab] Convert wallet context to ts.
Dec 5 2024, 21:08
PiRK accepted D17185: [chronik] Create a new optional JSON RPC interface.

It works with only a minor change in Electrum ABC code

Dec 5 2024, 20:58
teamcity added a comment to D17296: [cashtab] Cleanup theme file.
Dec 5 2024, 20:47
teamcity added a comment to D17296: [cashtab] Cleanup theme file.
Dec 5 2024, 20:45
johnkuney updated the diff for D17296: [cashtab] Cleanup theme file.

rebase

Dec 5 2024, 20:43
johnkuney requested review of D17296: [cashtab] Cleanup theme file.
Dec 5 2024, 20:41
bytesofman accepted D17294: [electrum] Better support for Trezor safe devices.
Dec 5 2024, 19:51
Fabien updated the diff for D17294: [electrum] Better support for Trezor safe devices.

Fix comment

Dec 5 2024, 19:32
Fabien updated the diff for D17185: [chronik] Create a new optional JSON RPC interface.

Sanitize the imports

Dec 5 2024, 19:30
Fabien accepted D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.

I still think it's not the best option but I'm not dying on the hill since both are valid anyway.

Dec 5 2024, 19:24
Fabien accepted D17290: [64-bit ints] Add script_tests for `getint` coverage.
Dec 5 2024, 19:19
bytesofman requested changes to D17294: [electrum] Better support for Trezor safe devices.
Dec 5 2024, 19:12
Fabien added inline comments to D17185: [chronik] Create a new optional JSON RPC interface.
Dec 5 2024, 18:38
Fabien added inline comments to D17294: [electrum] Better support for Trezor safe devices.
Dec 5 2024, 18:37
bytesofman added inline comments to D17294: [electrum] Better support for Trezor safe devices.
Dec 5 2024, 18:28
tobias_ruck requested changes to D17185: [chronik] Create a new optional JSON RPC interface.
Dec 5 2024, 18:05
tobias_ruck updated the diff for D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.

fix CheckNegate

Dec 5 2024, 17:27
tobias_ruck added inline comments to D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.
Dec 5 2024, 17:24
tobias_ruck closed D17000: [64-bit ints] Move `{Add,Sub}Int63Overflow(Emulated)` impl to cpp file.
Dec 5 2024, 17:16
tobias_ruck committed rABC805a0f137bf6: [64-bit ints] Move `{Add,Sub}Int63Overflow(Emulated)` impl to cpp file (authored by tobias_ruck).
[64-bit ints] Move `{Add,Sub}Int63Overflow(Emulated)` impl to cpp file
Dec 5 2024, 17:16
tobias_ruck updated the diff for D17290: [64-bit ints] Add script_tests for `getint` coverage.

add rationales

Dec 5 2024, 17:14
Fabien requested review of D17294: [electrum] Better support for Trezor safe devices.
Dec 5 2024, 15:45
bytesofman updated the diff for D17288: [Cashtab] Convert wallet context to ts.

typo patch

Dec 5 2024, 14:53
bytesofman added inline comments to D17288: [Cashtab] Convert wallet context to ts.
Dec 5 2024, 14:52
emack requested changes to D17288: [Cashtab] Convert wallet context to ts.
Dec 5 2024, 14:46
bytesofman published D17288: [Cashtab] Convert wallet context to ts for review.
Dec 5 2024, 14:19
bytesofman closed D17293: [ecash-herald] Tighten up daily summary msgs.
Dec 5 2024, 14:19
bytesofman committed rABCb5b7145ef776: [ecash-herald] Tighten up daily summary msgs (authored by bytesofman).
[ecash-herald] Tighten up daily summary msgs
Dec 5 2024, 14:19
emack accepted D17293: [ecash-herald] Tighten up daily summary msgs.
Dec 5 2024, 14:18
bytesofman published D17293: [ecash-herald] Tighten up daily summary msgs for review.
Dec 5 2024, 13:18
Fabien updated the diff for D17185: [chronik] Create a new optional JSON RPC interface.

Rebase, feedback

Dec 5 2024, 10:37
Fabien added inline comments to D17185: [chronik] Create a new optional JSON RPC interface.
Dec 5 2024, 10:36
Fabien accepted D17000: [64-bit ints] Move `{Add,Sub}Int63Overflow(Emulated)` impl to cpp file.
Dec 5 2024, 10:09
Fabien requested changes to D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.
Dec 5 2024, 10:06
Fabien requested changes to D17290: [64-bit ints] Add script_tests for `getint` coverage.
Dec 5 2024, 09:23
bytesofman closed D17291: [Cashtab] Remove swap screen.
Dec 5 2024, 00:20
bytesofman committed rABCd71229b11eca: [Cashtab] Remove swap screen (authored by bytesofman).
[Cashtab] Remove swap screen
Dec 5 2024, 00:20
emack accepted D17291: [Cashtab] Remove swap screen.

FWIW - The widget is working in prod for me

image.png (1×871 px, 66 KB)

Dec 5 2024, 00:19

Dec 4 2024

teamcity added a comment to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Dec 4 2024, 23:42
teamcity added a comment to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Dec 4 2024, 23:41
bytesofman updated the diff for D17269: [ecash-lib] Improve address handling of ecash dev libraries.

build lint

Dec 4 2024, 23:37
teamcity added a comment to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Dec 4 2024, 23:35
teamcity added a comment to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Dec 4 2024, 23:35
teamcity added a comment to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Dec 4 2024, 23:35
bytesofman added inline comments to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Dec 4 2024, 23:34
bytesofman updated the summary of D17291: [Cashtab] Remove swap screen.
Dec 4 2024, 23:34
bytesofman published D17291: [Cashtab] Remove swap screen for review.
Dec 4 2024, 23:33
bytesofman updated the diff for D17269: [ecash-lib] Improve address handling of ecash dev libraries.

improve file organization as suggested

Dec 4 2024, 23:33
tobias_ruck updated the summary of D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.
Dec 4 2024, 23:18
tobias_ruck updated the summary of D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.
Dec 4 2024, 23:06
tobias_ruck updated the diff for D16996: [64-bit ints] Change `CScriptNum::getint` to return `int64_t`.

address review, remove CScriptNum10 and replace tests with tests that test the behavior of CScriptNum instead of testing whether it matches CScriptNum10

Dec 4 2024, 23:02
tobias_ruck requested review of D17290: [64-bit ints] Add script_tests for `getint` coverage.
Dec 4 2024, 22:33
tobias_ruck updated the diff for D17000: [64-bit ints] Move `{Add,Sub}Int63Overflow(Emulated)` impl to cpp file.

address review

Dec 4 2024, 22:02
tobias_ruck added inline comments to D17185: [chronik] Create a new optional JSON RPC interface.
Dec 4 2024, 21:48
tobias_ruck requested changes to D17185: [chronik] Create a new optional JSON RPC interface.
Dec 4 2024, 21:45
tobias_ruck requested changes to D17269: [ecash-lib] Improve address handling of ecash dev libraries.
Dec 4 2024, 21:22
PiRK added inline comments to D17185: [chronik] Create a new optional JSON RPC interface.
Dec 4 2024, 20:24
Fabien requested review of D17185: [chronik] Create a new optional JSON RPC interface.
Dec 4 2024, 20:14
Fabien planned changes to D17185: [chronik] Create a new optional JSON RPC interface.
Dec 4 2024, 20:12
Fabien updated the diff for D17185: [chronik] Create a new optional JSON RPC interface.

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).

Dec 4 2024, 20:11
roqqit closed D17287: Remove executable file mode from some functional tests.
Dec 4 2024, 18:41
roqqit committed rABC518391383004: Remove executable file mode from some functional tests (authored by roqqit).
Remove executable file mode from some functional tests
Dec 4 2024, 18:41
teamcity added a comment to D17272: [CHRONIK][WEB][MODULES] Updated every out of date dependency in Cargo.toml that does not require code change.
Dec 4 2024, 14:06