Skip to content

Show balances of same account on other paratimes/consensus#2311

Merged
lukaw3d merged 8 commits into
masterfrom
lw/balance-on-other-layers
Nov 7, 2025
Merged

Show balances of same account on other paratimes/consensus#2311
lukaw3d merged 8 commits into
masterfrom
lw/balance-on-other-layers

Conversation

@lukaw3d

@lukaw3d lukaw3d commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Fixes #1858

Differences from design:

  • Runtime accounts do not have a total balance right now - showing available balance.
  • If available balance is zero but account has transactions I still show a link too, as user may be interested in account's history.
  • Simpler tooltip / not repeating the same address

https://pr-2311.oasis-explorer.pages.dev/mainnet/sapphire/address/oasis1qz78ap0456g2rk7j6rmtvasc9v2kjhz2s58qgj90#root
https://pr-2311.oasis-explorer.pages.dev/mainnet/sapphire/address/0x5d6D0A8fF8355Eb766B8d9Cec37d8e4313166564#root

Before | After
Screenshot from 2025-11-05 01-36-44

Tooltips
Recording 2025-11-05 at 01 39 48

Ignoring tooltip in tooltip weirdness:
Recording 2025-11-05 at 01 27 02

cc @donouwens

@github-actions

github-actions Bot commented Nov 5, 2025

Copy link
Copy Markdown

Deployed to Cloudflare Pages

Latest commit: 16756ca53545836168d680fc0dc1dfdecd5d6ed4
Status:✅ Deploy successful!
Preview URL: https://4f4ead40.oasis-explorer.pages.dev
Alias: https://pr-2311.oasis-explorer.pages.dev

@donouwens

donouwens commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Fixes #1858

Differences from design:

  • Runtime accounts do not have a total balance right now - showing available balance.

  • If available balance is zero but account has transactions I still show a link too, as user may be interested in account's history.

  • Simpler tooltip / not repeating the same address

....

cc @donouwens

All looks great to me. Great solution, thanks!

Should we inform user that there are transactions in the tooltip if there's no balance rather than saying 'n/a'?

@lukaw3d

lukaw3d commented Nov 5, 2025

Copy link
Copy Markdown
Contributor Author

Fixed comments from sync meeting

@lukaw3d lukaw3d force-pushed the lw/balance-on-other-layers branch from 550a210 to 1dd6c65 Compare November 6, 2025 02:21
@lukaw3d lukaw3d force-pushed the lw/balance-on-other-layers branch from 1dd6c65 to 16756ca Compare November 7, 2025 21:09
@lukaw3d lukaw3d merged commit 6758623 into master Nov 7, 2025
10 checks passed
@lukaw3d lukaw3d deleted the lw/balance-on-other-layers branch November 7, 2025 21:14
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.

[Feature]: When looking at an account, show that it also has balances on other paratimes and/or consensus and show a summary with a quick switcher.

3 participants