Unit tested helper function to get circulating supply
Unit tested helper function to get (net) burn or mint
Implement functions
Clean up rendering (spinner instead of "Loading..." ?)

Update: this diff is ok for implementing just circulating supply. Functions need to be designed with cache updates, but I think it's worth showing circulating supply before we get to that.

Tail of the build log:

Installing mock-chronik-client dependencies...
/work/modules/mock-chronik-client /work/abc-ci-builds/cashtab-tests

added 236 packages, and audited 237 packages in 2s

35 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
/work/cashtab /work/modules/mock-chronik-client /work/abc-ci-builds/cashtab-tests

added 1718 packages, and audited 1719 packages in 29s

262 packages are looking for funding
  run `npm fund` for details

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

> cashtab@2.15.1 build
> node scripts/build.js

Creating an optimized production build...
Failed to compile.

[eslint] 
src/components/Send/SendToken.js
  Line 43:5:  'undecimalizeTokenAmount' is defined but never used  no-unused-vars

Search for the keywords to learn more about each error.


Build cashtab-tests failed with exit code 1

• Unit tested helper function to get circulating supply
• Unit tested helper function to get (net) burn or mint
• Implement functions
• Clean up rendering (spinner instead of "Loading..." ?)

Are these coming into this diff or separately?

In D15847#358608, @emack wrote:

• Unit tested helper function to get circulating supply
• Unit tested helper function to get (net) burn or mint
• Implement functions
• Clean up rendering (spinner instead of "Loading..." ?)

Are these coming into this diff or separately?

oops

I think they should come separately. It will be a decent-sized refactor to get this working with the cache, and I won't know what that function should look like until I get into it.