Skip to content

refresh miner balance periodically and on mining new block#482

Merged
n13 merged 3 commits into
mainfrom
miner_refresh_balance
May 10, 2026
Merged

refresh miner balance periodically and on mining new block#482
n13 merged 3 commits into
mainfrom
miner_refresh_balance

Conversation

@n13

@n13 n13 commented May 9, 2026

Copy link
Copy Markdown
Collaborator

automatic balance refresh

  • Balance refresh is expensive - we query the chain for all nullifiers etc
  • Balance refreshes when we submit a block
  • Slight delay to allow network to update
  • Balance refreshes every 5 minutes in addition to that
  • Add logging for sanity
image

n13 added 2 commits May 9, 2026 23:19
miner needs to be run in release mode, otherwise very slow
@n13

n13 commented May 10, 2026

Copy link
Copy Markdown
Collaborator Author

Tested, this works... mined a block, balance updated

before/after

image image

Comment thread miner-app/lib/main.dart

@dewabisma dewabisma left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good to me, just wondering why we have that function call in main.dart

@dewabisma dewabisma left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@n13 n13 merged commit 469ace5 into main May 10, 2026
2 of 3 checks passed
dewabisma added a commit that referenced this pull request May 11, 2026
* wip: pos flow

- finish input charge screen
- finish scan to pay screen

* feat: finish pos screens, and formatting

* feat: make charge button big in bottom of screen

* chore: formatting

* refresh miner balance periodically and on mining new block (#482)

* refresh miner balance periodically and on mining new block

* miner run script

miner needs to be run in release mode, otherwise very slow

* add logging with rotating logs

* Miner Version 0.4.3

* Skeleton widget update (#480)

* feat: update skeleton widget with improved shimmer and colors

Rewrites Skeleton to use theme-aware skeletonHighlightA/B colors,
extracts TxItemSkeleton as a standalone widget, and converts
skeletonHighlight to two-stop skeletonHighlightA/B in AppColorsV2.

Co-authored-by: Cursor <cursoragent@cursor.com>

* feat: change skeleton to new one

* fix: PR review issues

- Dead V1 fields left behind.
- Opacity widget triggers a saveLayer()
- Tiny nit in TxItemSkeleton

---------

Co-authored-by: Cursor <cursoragent@cursor.com>

* feat: big clean up (#478)

* feat: update to new graphql

- comply with hasura syntax
- change to new graphql url in constants
- update data model construction from json to comply new hasura return value

* fix: revert to parse string

We have made hasura return string for number value.

* chore: formatting

* feat: finish mining reward screen

* feat: finish proper loading state

* feat: update empty state in home

* feat: big clean up

- Delete unused assets
- Delete unused widgets

* feat: update graphql wormhole to comply hasura

* feat: make planck as the highlighted mined stats and earned

* fix: query for transfer

change toHash to to_hash

* fix: import path

* feat: show current testnet details and redeem button

* feat: update skeleton widget with improved shimmer and colors

Rewrites Skeleton to use theme-aware skeletonHighlightA/B colors,
extracts TxItemSkeleton as a standalone widget, and converts
skeletonHighlight to two-stop skeletonHighlightA/B in AppColorsV2.

Co-authored-by: Cursor <cursoragent@cursor.com>

* revert: remove skeleton widget changes (moved to feat/skeleton-widget-update)

Co-authored-by: Cursor <cursoragent@cursor.com>

* feat: change skeleton to new one

* chore: formatting

---------

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Nikolaus Heger <nheger@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants