HomePhabricator

[Cashtab] Improvements to app handling of new blocks

Description

[Cashtab] Improvements to app handling of new blocks

Summary:
Now that we have in-node chronik, we should only be triggering app updates that are block related on avalanche-finalized blocks.

  1. The only app logic using blocks now is aliases. This should be gated by aliasSettings.aliasEnabled -- do this
  2. Add notifications for found blocks. May remove this after halving but might be a nice touch. I like the info.

Test Plan:
npm test

desktop:

image.png (84×330 px, 13 KB)

mobile:

image.png (125×380 px, 17 KB)

This diff is deployed at https://cashtab-local-dev.netlify.app/ for live testing

Reminder ancient bitcoin dev law: if you are actively testing something that requires waiting for a block, please expect unusually long block times

Reviewers: #bitcoin_abc, emack

Reviewed By: #bitcoin_abc, emack

Subscribers: emack

Differential Revision: https://reviews.bitcoinabc.org/D15889

Details

Provenance
bytesofmanAuthored on Apr 4 2024, 13:02
bytesofmanPushed on Apr 5 2024, 00:34
Reviewer
Restricted Project
Differential Revision
D15889: [Cashtab] Improvements to app handling of new blocks
Parents
rABC29de4acd6a4c: [Cashtab] Better label for show seed switch
Branches
Unknown
Tags
Unknown