Page MenuHomePhabricator

bytesofman (Joey)
UserAdministrator

Projects

User does not belong to any projects.

User Details

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

Recent Activity

Wed, May 15

bytesofman closed D16169: [token server + chronik-client + mock-chronik-client + ecash-herald] Remove legacy Dockerfile(s).
Wed, May 15, 19:58
bytesofman committed rABC69e7ae55530a: [token server + chronik-client + mock-chronik-client + ecash-herald] Remove… (authored by bytesofman).
[token server + chronik-client + mock-chronik-client + ecash-herald] Remove…
Wed, May 15, 19:58
bytesofman closed D16170: [ecash-lib] Update CI to also build ecashaddrjs now that chronik-client pulls this from the monorepo.
Wed, May 15, 19:58
bytesofman committed rABC9f74ce9542f8: [ecash-lib] Update CI to also build ecashaddrjs now that chronik-client pulls… (authored by bytesofman).
[ecash-lib] Update CI to also build ecashaddrjs now that chronik-client pulls…
Wed, May 15, 19:58
bytesofman published D16170: [ecash-lib] Update CI to also build ecashaddrjs now that chronik-client pulls this from the monorepo for review.
Wed, May 15, 16:35
bytesofman published D16169: [token server + chronik-client + mock-chronik-client + ecash-herald] Remove legacy Dockerfile(s) for review.
Wed, May 15, 16:31
bytesofman closed D16164: [ecash-herald] Finish deprecating unused API.
Wed, May 15, 16:19
bytesofman committed rABC56db4b23e951: [ecash-herald] Finish deprecating unused API (authored by bytesofman).
[ecash-herald] Finish deprecating unused API
Wed, May 15, 16:19
bytesofman added a comment to D16166: [e.cash] Mark upgrade complete in navbar.

@bot preview-e.cash

Wed, May 15, 13:32
bytesofman accepted D16165: [website] Show upgrade completed.
Wed, May 15, 13:25
bytesofman published D16164: [ecash-herald] Finish deprecating unused API for review.
Wed, May 15, 13:19
bytesofman closed D16135: [mock-chronik-client] use local deps.
Wed, May 15, 13:11
bytesofman committed rABCa65b5927af7b: [mock-chronik-client] use local deps (authored by bytesofman).
[mock-chronik-client] use local deps
Wed, May 15, 13:11
bytesofman updated the diff for D16135: [mock-chronik-client] use local deps.

rebase

Wed, May 15, 11:54
bytesofman closed D16160: [Cashtab] Show exact XEC amount sent or received in XEC tx history if user clicks tx.
Wed, May 15, 11:41
bytesofman committed rABC856394bfdae8: [Cashtab] Show exact XEC amount sent or received in XEC tx history if user… (authored by bytesofman).
[Cashtab] Show exact XEC amount sent or received in XEC tx history if user…
Wed, May 15, 11:41
bytesofman closed D16159: [Cashtab] Bump extension version to update published version to latest master.
Wed, May 15, 11:30
bytesofman committed rABC6ddb7dc2d245: [Cashtab] Bump extension version to update published version to latest master (authored by bytesofman).
[Cashtab] Bump extension version to update published version to latest master
Wed, May 15, 11:30
bytesofman closed D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.
Wed, May 15, 11:29
bytesofman committed rABC8d2fcbeccff9: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm (authored by bytesofman).
[chronik-client]+[alias-server]+[examples] Use local dep instead of npm
Wed, May 15, 11:29

Tue, May 14

bytesofman published D16160: [Cashtab] Show exact XEC amount sent or received in XEC tx history if user clicks tx for review.
Tue, May 14, 23:45
bytesofman published D16159: [Cashtab] Bump extension version to update published version to latest master for review.
Tue, May 14, 23:26
bytesofman added inline comments to D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.
Tue, May 14, 15:56
bytesofman closed D16138: [ecash-lib] Copy chronik to docker image.
Tue, May 14, 13:18
bytesofman committed rABC61d6e0071709: [ecash-lib] Copy chronik to docker image (authored by bytesofman).
[ecash-lib] Copy chronik to docker image
Tue, May 14, 13:18
bytesofman added inline comments to D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.
Tue, May 14, 13:11

Mon, May 13

bytesofman added a comment to D16138: [ecash-lib] Copy chronik to docker image.

You had the same test plan in D16131, how come it didn't work ? Did you run your test plan ?

Mon, May 13, 18:27
bytesofman updated the diff for D16138: [ecash-lib] Copy chronik to docker image.

make sure chronik/ files are in place before calling ./build-wasm

Mon, May 13, 18:25
bytesofman updated the diff for D16135: [mock-chronik-client] use local deps.

rebase

Mon, May 13, 18:09
bytesofman updated the diff for D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.

ecash-lib-integration tests also depend on ecashaddrjs

Mon, May 13, 17:51

Thu, May 9

bytesofman accepted D16111: [ecash-lib] Add agora advertisements (ads) of offers for AgoraOneshot.
Thu, May 9, 21:24
bytesofman planned changes to D16138: [ecash-lib] Copy chronik to docker image.

It is passing locally but so were the previous diffs

Thu, May 9, 19:24
bytesofman planned changes to D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.
Thu, May 9, 14:34
bytesofman planned changes to D16135: [mock-chronik-client] use local deps.

something going on with the rebasing here

Thu, May 9, 14:23
bytesofman updated the diff for D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.

lint

Thu, May 9, 14:22
bytesofman updated the diff for D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.

ecashaddrjs needs to be built first

Thu, May 9, 14:22
bytesofman retitled D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm from [chronik-client] Use local dep instead of npm to [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.
Thu, May 9, 14:19
bytesofman updated the diff for D16135: [mock-chronik-client] use local deps.

rebase to include updates from depends on

Thu, May 9, 14:18
bytesofman updated the diff for D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm.

update other apps that do not have CI deployments but do have CI tests

Thu, May 9, 14:16
bytesofman published D16134: [chronik-client]+[alias-server]+[examples] Use local dep instead of npm for review.
Thu, May 9, 13:48
bytesofman updated the diff for D16135: [mock-chronik-client] use local deps.

rebase to include js-mocha-integration-tests template update

Thu, May 9, 13:48
bytesofman published D16138: [ecash-lib] Copy chronik to docker image for review.
Thu, May 9, 13:11
bytesofman updated the diff for D16135: [mock-chronik-client] use local deps.

rebase

Thu, May 9, 12:51
bytesofman closed D16133: [token-server] Use monorepo hosted dependencies instead of npm.
Thu, May 9, 12:37
bytesofman committed rABC1513ebd3acf0: [token-server] Use monorepo hosted dependencies instead of npm (authored by bytesofman).
[token-server] Use monorepo hosted dependencies instead of npm
Thu, May 9, 12:37
bytesofman closed D16132: [ecash-herald] Use monorepo deps instead of npm.
Thu, May 9, 12:28
bytesofman committed rABCfd6cc31d1fbd: [ecash-herald] Use monorepo deps instead of npm (authored by bytesofman).
[ecash-herald] Use monorepo deps instead of npm
Thu, May 9, 12:28
bytesofman closed D16119: [chronik-client] Support websocket subscriptions to tokenId.
Thu, May 9, 12:25
bytesofman committed rABC179ea0249257: [chronik-client] Support websocket subscriptions to tokenId (authored by bytesofman).
[chronik-client] Support websocket subscriptions to tokenId
Thu, May 9, 12:25
bytesofman closed D16131: [ecash-lib] Add another stage to handle rust reqs for ecash-wasm build.
Thu, May 9, 12:14
bytesofman committed rABC676452b36ae9: [ecash-lib] Add another stage to handle rust reqs for ecash-wasm build (authored by bytesofman).
[ecash-lib] Add another stage to handle rust reqs for ecash-wasm build
Thu, May 9, 12:14

Wed, May 8

bytesofman published D16135: [mock-chronik-client] use local deps for review.
Wed, May 8, 22:49
bytesofman published D16133: [token-server] Use monorepo hosted dependencies instead of npm for review.
Wed, May 8, 22:05
bytesofman updated the diff for D16132: [ecash-herald] Use monorepo deps instead of npm.

remove cashtab copypasta, do not label stage 1 if there are no other stages

Wed, May 8, 22:04
bytesofman updated the test plan for D16132: [ecash-herald] Use monorepo deps instead of npm.
Wed, May 8, 21:50
bytesofman updated the diff for D16132: [ecash-herald] Use monorepo deps instead of npm.

update CI for local deps

Wed, May 8, 21:50
bytesofman planned changes to D16132: [ecash-herald] Use monorepo deps instead of npm.

need to update CI for tests

Wed, May 8, 21:44
bytesofman published D16132: [ecash-herald] Use monorepo deps instead of npm for review.
Wed, May 8, 21:44
bytesofman updated the diff for D16131: [ecash-lib] Add another stage to handle rust reqs for ecash-wasm build.

put Cargo.toml in place

Wed, May 8, 16:58
bytesofman published D16131: [ecash-lib] Add another stage to handle rust reqs for ecash-wasm build for review.
Wed, May 8, 16:55
bytesofman closed D16129: [Cashtab] Remove local legacy Dockerfile.
Wed, May 8, 13:22
bytesofman committed rABCd042703838b2: [Cashtab] Remove local legacy Dockerfile (authored by bytesofman).
[Cashtab] Remove local legacy Dockerfile
Wed, May 8, 13:22
bytesofman closed D16126: [ecash-lib] Dockerfile for CI publication to npm.
Wed, May 8, 13:17
bytesofman committed rABCf2211316c9fa: [ecash-lib] Dockerfile for CI publication to npm (authored by bytesofman).
[ecash-lib] Dockerfile for CI publication to npm
Wed, May 8, 13:17
bytesofman closed D16130: [Cashtab] Stop using Title for Tx History.
Wed, May 8, 13:16
bytesofman committed rABCa5fba94ae199: [Cashtab] Stop using Title for Tx History (authored by bytesofman).
[Cashtab] Stop using Title for Tx History
Wed, May 8, 13:16
bytesofman published D16130: [Cashtab] Stop using Title for Tx History for review.
Wed, May 8, 12:37
bytesofman updated the diff for D16126: [ecash-lib] Dockerfile for CI publication to npm.

improve comment order

Wed, May 8, 12:18
bytesofman updated the diff for D16126: [ecash-lib] Dockerfile for CI publication to npm.

remove checksum

Wed, May 8, 12:18
bytesofman closed D16112: [Cashtab] Set rewards eligibility to true on countdown expiration.
Wed, May 8, 12:16
bytesofman committed rABC63465698816e: [Cashtab] Set rewards eligibility to true on countdown expiration (authored by bytesofman).
[Cashtab] Set rewards eligibility to true on countdown expiration
Wed, May 8, 12:16
bytesofman published D16129: [Cashtab] Remove local legacy Dockerfile for review.
Wed, May 8, 12:15
bytesofman closed D16125: [Cashtab] Use monorepo dependencies instead of npm.
Wed, May 8, 12:07
bytesofman committed rABC051a73779c58: [Cashtab] Use monorepo dependencies instead of npm (authored by bytesofman).
[Cashtab] Use monorepo dependencies instead of npm
Wed, May 8, 12:07
bytesofman updated the diff for D16125: [Cashtab] Use monorepo dependencies instead of npm.

re-add legacy Dockerfile for now

Wed, May 8, 12:01
bytesofman added a comment to D16125: [Cashtab] Use monorepo dependencies instead of npm.

For the sake of avoiding breaking production, I would suggest you don't remove the cashtab/Dockerfile yet. This is the right move but it buys you time to update the CI to the new one and make sure everything is working before you actually remove the file, so you always have a backup.

Wed, May 8, 11:58
bytesofman accepted D16127: [gitian] Remove gitian.
Wed, May 8, 11:53
bytesofman accepted D16128: [doc] Minor improvments on the teamcity agent readme.
Wed, May 8, 11:51
bytesofman requested review of D16112: [Cashtab] Set rewards eligibility to true on countdown expiration.
Wed, May 8, 11:44
bytesofman closed D16123: [Cashtab] Debounce wallet select.
Wed, May 8, 03:40
bytesofman committed rABCca211dd7bc45: [Cashtab] Debounce wallet select (authored by bytesofman).
[Cashtab] Debounce wallet select
Wed, May 8, 03:40
bytesofman requested review of D16123: [Cashtab] Debounce wallet select.

We already have an implementation of this in the Alias codebase, what additional benefit do you get from using this library to achieve this?

Wed, May 8, 02:52

Tue, May 7

bytesofman published D16126: [ecash-lib] Dockerfile for CI publication to npm for review.
Tue, May 7, 22:56
bytesofman published D16125: [Cashtab] Use monorepo dependencies instead of npm for review.
Tue, May 7, 22:05
bytesofman published D16123: [Cashtab] Debounce wallet select for review.
Tue, May 7, 17:20
bytesofman added inline comments to D16112: [Cashtab] Set rewards eligibility to true on countdown expiration.
Tue, May 7, 16:46
bytesofman updated the diff for D16112: [Cashtab] Set rewards eligibility to true on countdown expiration.

improve logic by using padstart

Tue, May 7, 16:46
bytesofman closed D16122: [Cashtab] Improve minified nav header to prevent content jumping.
Tue, May 7, 16:33
bytesofman committed rABCf16bbdb8b284: [Cashtab] Improve minified nav header to prevent content jumping (authored by bytesofman).
[Cashtab] Improve minified nav header to prevent content jumping
Tue, May 7, 16:33
bytesofman added a comment to D16122: [Cashtab] Improve minified nav header to prevent content jumping.

Much better. Although I noticed it is scrolling even when there isn't content down below, however that's in prod too so not specific to this diff but nontheless I think previously we tried to stop that?

Tue, May 7, 16:31

Mon, May 6

bytesofman published D16122: [Cashtab] Improve minified nav header to prevent content jumping for review.
Mon, May 6, 22:52
bytesofman requested changes to D16111: [ecash-lib] Add agora advertisements (ads) of offers for AgoraOneshot.

It is a bit complicated to setup these txs. Not too big of a deal since will be handled by devs and not users.

Mon, May 6, 21:30
bytesofman published D16119: [chronik-client] Support websocket subscriptions to tokenId for review.
Mon, May 6, 19:49
bytesofman closed D16088: [chronik-client] Support lokad id endpoints.
Mon, May 6, 17:23
bytesofman committed rABC893e80e4a44a: [chronik-client] Support lokad id endpoints (authored by bytesofman).
[chronik-client] Support lokad id endpoints
Mon, May 6, 17:23
bytesofman added inline comments to D16112: [Cashtab] Set rewards eligibility to true on countdown expiration.
Mon, May 6, 17:09
bytesofman updated the diff for D16112: [Cashtab] Set rewards eligibility to true on countdown expiration.

use const for repeated elapsed timers, correct double 0 string issue

Mon, May 6, 17:09
bytesofman added inline comments to D16088: [chronik-client] Support lokad id endpoints.
Mon, May 6, 16:56
bytesofman updated the diff for D16088: [chronik-client] Support lokad id endpoints.

wait for expected ws msgs before testing them

Mon, May 6, 16:55
bytesofman planned changes to D16088: [chronik-client] Support lokad id endpoints.

Failed tests logs:

====== History endpoints and websocket for LOKAD ID: Websocket and endpoints correctly recognize txs with a lokadId in an input scriptSig and the same lokadId in OP_RETURN (non-EMPP).History endpoints and websocket for LOKAD ID Websocket and endpoints correctly recognize txs with a lokadId in an input scriptSig and the same lokadId in OP_RETURN (non-EMPP) ======
AssertionError: expected [] to deeply equal [ { type: 'Tx', …(2) } ]
    at /work/modules/chronik-client/test/integration/lokad_id.ts:254:47
    at Generator.next (<anonymous>)
    at fulfilled (test/integration/lokad_id.ts:31:58)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

      + expected - actual

      -[]
      +[
      +  {
      +    "msgType": "TX_ADDED_TO_MEMPOOL"
      +    "txid": "1ef4bebdb5a1298877aed9b7f741e3e0903820514316a35f3e7c05957308471a"
      +    "type": "Tx"
      +  }
      +]
====== History endpoints and websocket for LOKAD ID: Websocket and endpoints correctly recognize a tx with a new lokadId in a spend input and a new lokad id in EMPP OP_RETURN.History endpoints and websocket for LOKAD ID Websocket and endpoints correctly recognize a tx with a new lokadId in a spend input and a new lokad id in EMPP OP_RETURN ======
AssertionError: expected [ { type: 'Tx', …(2) }, …(1) ] to deeply equal [ { type: 'Tx', …(2) } ]
    at /work/modules/chronik-client/test/integration/lokad_id.ts:307:47
    at Generator.next (<anonymous>)
    at fulfilled (test/integration/lokad_id.ts:31:58)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

      + expected - actual

       [
         {
           "msgType": "TX_ADDED_TO_MEMPOOL"
      -    "txid": "1ef4bebdb5a1298877aed9b7f741e3e0903820514316a35f3e7c05957308471a"
      -    "type": "Tx"
      -  }
      -  {
      -    "msgType": "TX_ADDED_TO_MEMPOOL"
           "txid": "a989e07ae47f3925e911942e6dfbb0a5217fbd6617975b5c7a03a235c0c8a554"
           "type": "Tx"
         }
       ]
====== History endpoints and websocket for LOKAD ID: After these txs are mined, chronik-client returns this info appropriately.History endpoints and websocket for LOKAD ID After these txs are mined, chronik-client returns this info appropriately ======
AssertionError: expected [ { type: 'Tx', …(2) }, …(1) ] to deeply equal [ { type: 'Tx', …(2) } ]
    at /work/modules/chronik-client/test/integration/lokad_id.ts:372:59
    at Generator.next (<anonymous>)
    at fulfilled (test/integration/lokad_id.ts:31:58)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

      + expected - actual

           "msgType": "TX_CONFIRMED"
           "txid": "1ef4bebdb5a1298877aed9b7f741e3e0903820514316a35f3e7c05957308471a"
           "type": "Tx"
         }
      -  {
      -    "msgType": "TX_CONFIRMED"
      -    "txid": "a989e07ae47f3925e911942e6dfbb0a5217fbd6617975b5c7a03a235c0c8a554"
      -    "type": "Tx"
      -  }
       ]

Each failure log is accessible here:
History endpoints and websocket for LOKAD ID: Websocket and endpoints correctly recognize txs with a lokadId in an input scriptSig and the same lokadId in OP_RETURN (non-EMPP).History endpoints and websocket for LOKAD ID Websocket and endpoints correctly recognize txs with a lokadId in an input scriptSig and the same lokadId in OP_RETURN (non-EMPP)
History endpoints and websocket for LOKAD ID: Websocket and endpoints correctly recognize a tx with a new lokadId in a spend input and a new lokad id in EMPP OP_RETURN.History endpoints and websocket for LOKAD ID Websocket and endpoints correctly recognize a tx with a new lokadId in a spend input and a new lokad id in EMPP OP_RETURN
History endpoints and websocket for LOKAD ID: After these txs are mined, chronik-client returns this info appropriately.History endpoints and websocket for LOKAD ID After these txs are mined, chronik-client returns this info appropriately

Mon, May 6, 16:33