Page MenuHomePhabricator

bytesofman (Joey)
UserAdministrator

Projects

User does not belong to any projects.

User Details

User Since
Nov 5 2020, 18:28 (229 w, 5 d)
Roles
Administrator

Recent Activity

Yesterday

bytesofman accepted D17871: [ecash-lib] Add `parseSlp`.

green w/ nits on remaining _LEN constants

Tue, Apr 1, 13:22

Mon, Mar 31

bytesofman closed D17876: [Cashtab] Streamline loading conditions in flaky test.
Mon, Mar 31, 21:25
bytesofman committed rABC16124b0c2cfc: [Cashtab] Streamline loading conditions in flaky test (authored by bytesofman).
[Cashtab] Streamline loading conditions in flaky test
Mon, Mar 31, 21:25
bytesofman published D17876: [Cashtab] Streamline loading conditions in flaky test for review.
Mon, Mar 31, 16:34

Sun, Mar 30

bytesofman closed D17875: [token-server] Bump recaptcha score.
Sun, Mar 30, 23:58
bytesofman committed rABC2cba279342f5: [token-server] Bump recaptcha score (authored by bytesofman).
[token-server] Bump recaptcha score
Sun, Mar 30, 23:58
bytesofman published D17875: [token-server] Bump recaptcha score for review.
Sun, Mar 30, 13:49

Sat, Mar 29

bytesofman added inline comments to D17871: [ecash-lib] Add `parseSlp`.
Sat, Mar 29, 04:07

Fri, Mar 28

bytesofman added a comment to D17871: [ecash-lib] Add `parseSlp`.

Is it possible to have a "valid" SLP OP_RETURN string that exceeds the OP_RETURN byte limit?

Fri, Mar 28, 21:43
bytesofman requested changes to D17871: [ecash-lib] Add `parseSlp`.

How is this function expected to be used? For the most part, correctly parsing an SLP OP_RETURN is often irrelevant, since we lack the indexer data (e.g. a well-formed SEND OP_RETURN could be a burn; validity in general depends on the indexer).

Fri, Mar 28, 17:29

Tue, Mar 25

bytesofman committed rABCceb4df3ac043: [Cashtab] Show the normal list workflow for XECX from the list option (authored by bytesofman).
[Cashtab] Show the normal list workflow for XECX from the list option
Tue, Mar 25, 12:08
bytesofman closed D17854: [Cashtab] Show the normal list workflow for XECX from the list option.
Tue, Mar 25, 12:08
bytesofman updated the test plan for D17854: [Cashtab] Show the normal list workflow for XECX from the list option.
Tue, Mar 25, 04:56
bytesofman updated the diff for D17854: [Cashtab] Show the normal list workflow for XECX from the list option.

patch conditional rendering for test

Tue, Mar 25, 04:53
bytesofman published D17854: [Cashtab] Show the normal list workflow for XECX from the list option for review.
Tue, Mar 25, 04:46

Mon, Mar 24

bytesofman planned changes to D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.

Pushing up this last round of feedback-based changes to support incremental review, but this is not ready to land yet

Mon, Mar 24, 21:26
bytesofman updated the diff for D17822: [ecash-wallet] Add fluent API for building and broadcasting txs.

Update interfaces, add some intermediate steps, take sighash as a param

Mon, Mar 24, 21:23

Fri, Mar 21

bytesofman published D17822: [ecash-wallet] Add fluent API for building and broadcasting txs for review.
Fri, Mar 21, 03:17
bytesofman committed rABC5b53dd1d5d01: [ecash-herald] Improve miner parsing (authored by bytesofman).
[ecash-herald] Improve miner parsing
Fri, Mar 21, 03:13
bytesofman closed D17841: [ecash-herald] Improve miner parsing.
Fri, Mar 21, 03:13

Thu, Mar 20

bytesofman published D17841: [ecash-herald] Improve miner parsing for review.
Thu, Mar 20, 22:57
bytesofman requested changes to D17815: [Cashtab] Introduce CSS variables to make styling easier.

this diff was created on the master branch, which makes it difficult to rebase onto the latest master

Thu, Mar 20, 01:15

Wed, Mar 19

bytesofman accepted D17830: Bump the log crawling timeout after node restart in rpc_setban.py.
Wed, Mar 19, 20:51
bytesofman added a comment to D17815: [Cashtab] Introduce CSS variables to make styling easier.
  • If we are switching from px to rem, switch; remove all the px comments, mb keep one to show what the base-level px is
  • do not use calc. if some ratios seem unreadable, create a variable for them and use the variable
  • remove all unused variables
Wed, Mar 19, 14:46
bytesofman closed D17826: [install readme] Add deps needed to build with chronik and plugins.
Wed, Mar 19, 14:42
bytesofman committed rABC0ba12c26bb6d: [install readme] Add deps needed to build with chronik and plugins (authored by bytesofman).
[install readme] Add deps needed to build with chronik and plugins
Wed, Mar 19, 14:42
bytesofman updated the diff for D17826: [install readme] Add deps needed to build with chronik and plugins.

back out the pythonpath stuff, add dep

Wed, Mar 19, 14:25
bytesofman updated the diff for D17826: [install readme] Add deps needed to build with chronik and plugins.

debian python note

Wed, Mar 19, 13:58
bytesofman published D17826: [install readme] Add deps needed to build with chronik and plugins for review.
Wed, Mar 19, 13:56
bytesofman closed D17821: [Cashtab] Patch bug in rendering of managed NFTs.
Wed, Mar 19, 12:26
bytesofman committed rABCb50acc96032b: [Cashtab] Patch bug in rendering of managed NFTs (authored by bytesofman).
[Cashtab] Patch bug in rendering of managed NFTs
Wed, Mar 19, 12:26
bytesofman added inline comments to D17815: [Cashtab] Introduce CSS variables to make styling easier.
Wed, Mar 19, 03:28
bytesofman published D17821: [Cashtab] Patch bug in rendering of managed NFTs for review.
Wed, Mar 19, 03:18

Tue, Mar 18

bytesofman accepted D17820: [CI] Fix the chronik-plugins docker image.
Tue, Mar 18, 23:29
bytesofman requested changes to D17815: [Cashtab] Introduce CSS variables to make styling easier.

would be a good change and help to get some of the css more standardized in Cashtab

Tue, Mar 18, 20:09
bytesofman closed D17814: [chronik-client] Patch failoverProxy to recognize another type of server failure.
Tue, Mar 18, 19:05
bytesofman committed rABCcdbb66df256c: [chronik-client] Patch failoverProxy to recognize another type of server failure (authored by bytesofman).
[chronik-client] Patch failoverProxy to recognize another type of server failure
Tue, Mar 18, 19:05
bytesofman updated the diff for D17814: [chronik-client] Patch failoverProxy to recognize another type of server failure.

better comments

Tue, Mar 18, 18:34
bytesofman updated the test plan for D17814: [chronik-client] Patch failoverProxy to recognize another type of server failure.
Tue, Mar 18, 04:05

Mon, Mar 17

bytesofman published D17814: [chronik-client] Patch failoverProxy to recognize another type of server failure for review.
Mon, Mar 17, 23:50
bytesofman closed D17812: [Cashtab] Move native2 to top.
Mon, Mar 17, 05:04
bytesofman committed rABC53526c84d87f: [Cashtab] Move native2 to top (authored by bytesofman).
[Cashtab] Move native2 to top
Mon, Mar 17, 05:04
bytesofman published D17812: [Cashtab] Move native2 to top for review.
Mon, Mar 17, 05:03

Sun, Mar 16

bytesofman accepted D17774: [ecash-lib-wasm] Add readme with installation instructions.
Sun, Mar 16, 13:19

Sat, Mar 15

bytesofman closed D17811: [ecash-wallet] Move wallet class code and tests to wallet files, use an indext file for exports.
Sat, Mar 15, 23:03
bytesofman committed rABC38169309a7b3: [ecash-wallet] Move wallet class code and tests to wallet files, use an indext… (authored by bytesofman).
[ecash-wallet] Move wallet class code and tests to wallet files, use an indext…
Sat, Mar 15, 23:03
bytesofman accepted D17806: [electrum] Make the coin consolidator not burn alp tokens.
Sat, Mar 15, 22:49

Fri, Mar 14

bytesofman published D17811: [ecash-wallet] Move wallet class code and tests to wallet files, use an indext file for exports for review.
Fri, Mar 14, 21:47
bytesofman closed D17779: [ecash-wallet] Add method to get spendable sats utxos.
Fri, Mar 14, 21:41
bytesofman committed rABCd0ecde3d1500: [ecash-wallet] Add method to get spendable sats utxos (authored by bytesofman).
[ecash-wallet] Add method to get spendable sats utxos
Fri, Mar 14, 21:41
bytesofman updated the diff for D17779: [ecash-wallet] Add method to get spendable sats utxos.

tighten up comments, remove "get" prefix from utxo selection methods

Fri, Mar 14, 21:04
bytesofman added inline comments to D17779: [ecash-wallet] Add method to get spendable sats utxos.
Fri, Mar 14, 21:03
bytesofman added a comment to D17806: [electrum] Make the coin consolidator not burn alp tokens.

TIL coin consolidator is a thing in electrum

Fri, Mar 14, 17:38
bytesofman abandoned D17802: [Cashtab] Correct issue of two tests with the same name.

another diff just deleted the test

Fri, Mar 14, 17:28
bytesofman closed D17807: [token-server] Soften recaptcha check.
Fri, Mar 14, 16:01
bytesofman committed rABC459f2bd542e5: [token-server] Soften recaptcha check (authored by bytesofman).
[token-server] Soften recaptcha check
Fri, Mar 14, 16:01
bytesofman updated the diff for D17779: [ecash-wallet] Add method to get spendable sats utxos.

rebase

Fri, Mar 14, 13:43
bytesofman retitled D17779: [ecash-wallet] Add method to get spendable sats utxos from [ecash-wallet] Add method to get fuel utxos, i.e. utxos used to send eCash or fuel token txs to [ecash-wallet] Add method to get spendable sats utxos.
Fri, Mar 14, 13:39
bytesofman published D17807: [token-server] Soften recaptcha check for review.
Fri, Mar 14, 13:33
bytesofman closed D17804: [Cashtab] Use ecash-lib for HDNode.
Fri, Mar 14, 13:33
bytesofman committed rABC72d6eb29573c: [Cashtab] Use ecash-lib for HDNode (authored by bytesofman).
[Cashtab] Use ecash-lib for HDNode
Fri, Mar 14, 13:33
bytesofman requested changes to D17774: [ecash-lib-wasm] Add readme with installation instructions.
Fri, Mar 14, 13:27

Thu, Mar 13

bytesofman published D17804: [Cashtab] Use ecash-lib for HDNode for review.
Thu, Mar 13, 23:36
bytesofman closed D17778: [Cashtab] Use ecash-lib to sign and verify msgs.
Thu, Mar 13, 22:40
bytesofman committed rABC5c1b0068bb80: [Cashtab] Use ecash-lib to sign and verify msgs (authored by bytesofman).
[Cashtab] Use ecash-lib to sign and verify msgs
Thu, Mar 13, 22:40
bytesofman closed D17803: [Cashtab] Drop test causing problems in CI.
Thu, Mar 13, 22:10
bytesofman committed rABCcf49ef6cf520: [Cashtab] Drop test causing problems in CI (authored by bytesofman).
[Cashtab] Drop test causing problems in CI
Thu, Mar 13, 22:10
bytesofman updated the diff for D17779: [ecash-wallet] Add method to get spendable sats utxos.

rebase, correct indefinite article in test name

Thu, Mar 13, 22:09
bytesofman updated the diff for D17803: [Cashtab] Drop test causing problems in CI.

remove other test that appears to fail for the same reason

Thu, Mar 13, 21:05
bytesofman published D17803: [Cashtab] Drop test causing problems in CI for review.
Thu, Mar 13, 21:01
bytesofman requested review of D17778: [Cashtab] Use ecash-lib to sign and verify msgs.
Thu, Mar 13, 17:05
bytesofman published D17802: [Cashtab] Correct issue of two tests with the same name for review.
Thu, Mar 13, 16:26
bytesofman updated the diff for D17779: [ecash-wallet] Add method to get spendable sats utxos.

use spendablesatsonlyutxos instead of fuelutxos, make getSpendableCoinbaseUtxos private

Thu, Mar 13, 16:25
bytesofman added inline comments to D17779: [ecash-wallet] Add method to get spendable sats utxos.
Thu, Mar 13, 16:24
bytesofman closed D17782: [ecash-lib] Allow use of .only in mocha test files.
Thu, Mar 13, 14:08
bytesofman committed rABC6f6281853271: [ecash-lib] Allow use of .only in mocha test files (authored by bytesofman).
[ecash-lib] Allow use of .only in mocha test files
Thu, Mar 13, 14:08

Wed, Mar 12

bytesofman closed D17794: [Cashtab] Add POW to agora landing page.
Wed, Mar 12, 21:24
bytesofman committed rABC0bb8a45a6dd6: [Cashtab] Add POW to agora landing page (authored by bytesofman).
[Cashtab] Add POW to agora landing page
Wed, Mar 12, 21:24
bytesofman published D17794: [Cashtab] Add POW to agora landing page for review.
Wed, Mar 12, 21:11
bytesofman closed D17786: [Cashtab] Patch header links to token page in extension.
Wed, Mar 12, 20:52
bytesofman committed rABCc00fa844c161: [Cashtab] Patch header links to token page in extension (authored by bytesofman).
[Cashtab] Patch header links to token page in extension
Wed, Mar 12, 20:52
bytesofman updated the diff for D17778: [Cashtab] Use ecash-lib to sign and verify msgs.

add comments for verifyMsg

Wed, Mar 12, 20:24
bytesofman added inline comments to D17778: [Cashtab] Use ecash-lib to sign and verify msgs.
Wed, Mar 12, 20:24
bytesofman published D17786: [Cashtab] Patch header links to token page in extension for review.
Wed, Mar 12, 17:23
bytesofman closed D17775: [Cashtab] Show XECX sweeper balance and notice if user redemption will not be instant.
Wed, Mar 12, 16:44
bytesofman committed rABC212529d453fa: [Cashtab] Show XECX sweeper balance and notice if user redemption will not be… (authored by bytesofman).
[Cashtab] Show XECX sweeper balance and notice if user redemption will not be…
Wed, Mar 12, 16:44
bytesofman requested review of D17782: [ecash-lib] Allow use of .only in mocha test files.

Add .only to a test file in ecash-lib, run arc lint, get a lint error, remove it, all good, npm test with a test using .only and only that test runs

I added .only to describe('SLP Integration Test') and it is not the only test that is run.

image.png (840×692 px, 96 KB)

Wed, Mar 12, 16:19
bytesofman added inline comments to D17778: [Cashtab] Use ecash-lib to sign and verify msgs.
Wed, Mar 12, 03:50
bytesofman requested review of D17775: [Cashtab] Show XECX sweeper balance and notice if user redemption will not be instant.
Wed, Mar 12, 03:27

Tue, Mar 11

bytesofman published D17782: [ecash-lib] Allow use of .only in mocha test files for review.
Tue, Mar 11, 23:11
bytesofman published D17778: [Cashtab] Use ecash-lib to sign and verify msgs for review.
Tue, Mar 11, 23:10
bytesofman accepted D17781: [chronik] Bump corrosion to 0.5.1.
Tue, Mar 11, 21:12
bytesofman accepted D17780: [ci] Fix wasm install with newer rust versions.
Tue, Mar 11, 21:12
bytesofman updated the diff for D17779: [ecash-wallet] Add method to get spendable sats utxos.

no arrow syntax, stop testing files that do not yet exist, use correct COINBASE_MATURITY term, clean up comments

Tue, Mar 11, 20:34
bytesofman updated the diff for D17779: [ecash-wallet] Add method to get spendable sats utxos.

do not assume coinbase utxos cannot be tokens, add token utxos to tests to show we do not get them as fuel utxos

Tue, Mar 11, 18:27
bytesofman published D17779: [ecash-wallet] Add method to get spendable sats utxos for review.
Tue, Mar 11, 18:16
bytesofman closed D17776: [ecash-agora] Support full-accept agora partials.
Tue, Mar 11, 16:15
bytesofman committed rABCc92a2d2520d0: [ecash-agora] Support full-accept agora partials (authored by bytesofman).
[ecash-agora] Support full-accept agora partials
Tue, Mar 11, 16:15
bytesofman accepted D17756: [Cashtab] Add click-outside-area closing effect for NavWrapper using event bubbling.
Tue, Mar 11, 15:10
bytesofman published D17776: [ecash-agora] Support full-accept agora partials for review.
Tue, Mar 11, 15:08