In D17782#405166, @emack wrote:

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.

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

minor UI improvement (change the greyed out spend action text to tell why the coin is unspendable)

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

Some initial comments

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

An attempt to resolve a missing base commit

otherwise the code is fine, diff is ready to roll with exception of mystery git basis / arc patch issue

mb it's not the best git-fu but in my experience it's almost always easier to recreate a diff with a single commit when phab/git raise issues (vs attempt to squash / update all the necessary arcanist stuff...who knows what's in there or what it was originally based on).

looks like this is not based on the latest master or is accidentally a stack

Tested all ok across chrome and firefox.

fix it description

• add test for all root certs
• fix handling of NULL parameters in readAlgIdent
• err on invalid date instead of setting it to 0

remove another XecWallet mention

more fixes from the rename and constructor change, better property comments

remove redundant interface, change name to Wallet, private constructor with fromSk static method helper constructor

[Cashtab] Add click-outside-area closing effect for NavWrapper using event bubbling

lint

remove stopPropagation

Updating D17756: [Cashtab] Add click-outside-area closing effect for NavWrapper using event bubbling

In D17767#404783, @Kronkmeister wrote:

Not sure why the builds failed and the bot doesn't respond. The change works on my end. Lints say there's nothing to lint (since it's just updating an image file).

Build Bitcoin ABC Diffs / Diff Testing (preview-e.cash) passed.
Preview is available at http://51.178.130.230:41749 for the next 60 minutes.

Not sure why the builds failed and the bot doesn't respond. The change works on my end. Lints say there's nothing to lint (since it's just updating an image file).

@bot preview-e.cash