Page MenuHomePhabricator
Feed Advanced Search

Tue, Apr 8

bytesofman requested changes to D17913: [chronik-client] Added a mechanism for automatically selecting the fastest responding node, supporting two strategies: AsOrdered and ClosestFirst..

without benchmarking this is difficult to review. you may need to run this as a custom version on some apps and see how it behaves in prod.

Tue, Apr 8, 23:48

Mon, Apr 7

bytesofman updated the diff for D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.

update missed test function name

Mon, Apr 7, 23:42
bytesofman updated the diff for D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.

add TODO for OP_RETURN parsing, update and export actions, use correct order for actions, typo patch

Mon, Apr 7, 22:43
bytesofman added inline comments to D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.
Mon, Apr 7, 22:42

Sat, Apr 5

bytesofman accepted D17903: [Cashtab] Add 2 chronik nodes.
Sat, Apr 5, 22:53

Fri, Apr 4

bytesofman accepted D17900: [ecash-lib] Add `parseAlp`.
Fri, Apr 4, 20:48
bytesofman requested changes to D17900: [ecash-lib] Add `parseAlp`.
Fri, Apr 4, 18:24

Thu, Apr 3

bytesofman added a comment to D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.

NB will need to add more tests but needs review now to confirm API and dev direction ... extensively testing the wrong API won't help anyone

Thu, Apr 3, 21:06
bytesofman updated the diff for D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.

rebase

Thu, Apr 3, 21:06
bytesofman updated the diff for D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.

remove dead code

Thu, Apr 3, 21:05