Skip to content

[explorer/frontend] feat: add harvest state section - #2393

Open
AnthonyLaw wants to merge 3 commits into
devfrom
explorer/frontend-add-harvest-status
Open

AnthonyLaw wants to merge 3 commits into
devfrom
explorer/frontend-add-harvest-status

Conversation

@AnthonyLaw

Copy link
Copy Markdown
Member

Problem: Missing harvest state information from account pages. #2357
Solution: Show harvest state when harvested blocks > 0

image

@AnthonyLaw
AnthonyLaw requested a review from OlegMakarenko July 12, 2026 11:42
@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.00%. Comparing base (e882c42) to head (ef67c39).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #2393      +/-   ##
==========================================
+ Coverage   95.57%   97.00%   +1.42%     
==========================================
  Files         661      276     -385     
  Lines       51522    21335   -30187     
  Branches     1362      208    -1154     
==========================================
- Hits        49244    20695   -28549     
+ Misses       2230      634    -1596     
+ Partials       48        6      -42     
Flag Coverage Δ
explorer-frontend 94.82% <100.00%> (-0.06%) ⬇️
explorer-puller ?
explorer-rest ?
wallet-mobile-symbol ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
explorer/frontend/api/accounts.js 100.00% <100.00%> (ø)
explorer/frontend/pages/accounts/[address].jsx 98.40% <100.00%> (-1.26%) ⬇️

... and 385 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AnthonyLaw
AnthonyLaw force-pushed the explorer/frontend-add-harvest-status branch from e50461f to ef67c39 Compare July 12, 2026 11:59
@AnthonyLaw
AnthonyLaw requested a review from Wayonb July 12, 2026 11:59

@OlegMakarenko OlegMakarenko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The sections should not change each other width. Also I would better move harvesting into the history section with tabs: Transactions and Harvested for displaying the list of received rewards. If we really need to present the total harvested amount, I would move it into a new statistics section or tab.

Let me design it first.

@cryptoBeliever

Copy link
Copy Markdown
Contributor

@OlegMakarenko, here are a few ideas for adding a harvested blocklist and statistics. Maybe one of them will be a good fit with a bit of fine-tuning.

W1-desktop W1-mobile
W2-desktop W2-mobile
W3-desktop W3-mobile
W4-desktop W4-mobile
W5-desktop W5-mobile

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.

3 participants