cartesi / explorer

Cartesi Blockchain Explorer
https://explorer.cartesi.io
Apache License 2.0
9 stars 12 forks source link

#60 Update blocks page UI #95

Closed nevendyulgerov closed 1 year ago

nevendyulgerov commented 1 year ago

Uses as base branch feature/57-update-stake-page-ui (PR).

This is how the blocks page looks after the changes: https://github.com/cartesi/explorer/assets/6005179/56ba783d-5629-4402-9006-b0e1727d431e

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rollups ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 13, 2023 6:54pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 13, 2023 6:54pm
coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5818393143

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/staking/src/components/BlocksChart.tsx 0 4 0.0%
<!-- Total: 5 9 55.56% -->
Files with Coverage Reduction New Missed Lines %
apps/staking/src/components/stake/tables/PoolPerformanceTableRow.tsx 5 73.33%
<!-- Total: 5 -->
Totals Coverage Status
Change from base Build 5812043593: 0.09%
Covered Lines: 2654
Relevant Lines: 3451

💛 - Coveralls
brunomenezes commented 1 year ago

@nevendyulgerov, That is still in draft. But the build is broken, and also I am seeing merge commits 😮. Maybe a squash here and rebase from epic/52-theme-update-new-dark-mode would be good.

brunomenezes commented 1 year ago

@nevendyulgerov, I merged the https://github.com/cartesi/explorer/pull/80 which is the base branch you use here. Could you rebase and fix the conflicts? Probably you will need to do the same with #102 since it is a "waterfall" approach here.