One nit

Otherwise lgtm

Otherwise seems good to go, the timeFirstSeen seems fine.

Good catch !

still need to fix the decompress_script injection which currently breaks tests

Actually I'm not sure we should remove it. It seems like this was reused and re-removed even after D4576 (see D5283). Maybe we should keep looking for it in Electrum it for another potential future node consensus upgrade?

rebase

rebase

remove now useless functions and imports

remove the need to access db for checking if the index is enabled

no more need for the todo

then_some, don't allocate

revert accidental squashing

I accidentaly squashed the child diff while trying to reabase and amend it

use then_some, don't allocate to query the hashmap

return a better error if the scripthash index is disabled

improve spec description in line with suggested comments, provide spec for Cashtab planned implementation

handle invalid tokenId, handle /token/ route, confirm with test

http://localhost:3000/#/token/ just renders a blank page with the header - perhaps default to / route?.

We might also want to use Shammah's CashWeb, which is much more generic and powerful, but also creates much less data on-chain:

http://localhost:3000/#/token/ just renders a blank page with the header - perhaps default to / route?.

back out unrelated lint change

rebase, split out validation into public method for partials, preventUnacceptableRemainder

EMPP for trollbox spec, add version byte, add msg type byte, update example

looks like the decompress_script injection breaks a bunch of tests

back out unused and untested negative number logic

In D17044#386646, @emack wrote:

Is this because the quantity exceeds circulating supply? Can't you end the slider range at 745220833808901734.4?