Skip to content

Cleanup react #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 15 commits into from
Mar 11, 2025
Merged

Cleanup react #13

merged 15 commits into from
Mar 11, 2025

Conversation

@patrick-ogrady
Copy link
Contributor

@patrick-ogrady patrick-ogrady commented Mar 11, 2025 •
edited
Loading

TODO

  • Don't show time if negative
  • Zoom out on map slightly

@patrick-ogrady
commit wip
fe01e9d
@patrick-ogrady
Map bars (#11)
76a2e09 
* progress

* growing bars

* much closer

* fix timeout issue

* layering bars

* zoomed in

* layering correct

* below bar

* getting closer

* very close

* change hex formatting

* don't emit skipped

* shorten hex

* handle no time

* zoom in

* remove stat card

* remove timed out text

* fix seed by 1

* fix multi-view tracking

* dumb interface

* better mobile

* responsive scaling works

* Working

* Commonware aesthetic (#12)

* mirror commonware aesthetic

* nits

* remove connected

* fix legend colors

* pretty good

* make circle a little bigger

* good
@patrick-ogrady
nits
532387c
@patrick-ogrady
unify locations
66dbc08
@patrick-ogrady
Seed for leader (#14)
f363717 
* progress

* fix seed selection

* add network key

* fix legend
@patrick-ogrady
update README
22ddec7
@patrick-ogrady
fix zero issue
8f6611f
@patrick-ogrady
update zoom to 1
2fd6b20
@patrick-ogrady
remove junk
dbb8d34
@patrick-ogrady
fix build errors
6d4e857
@patrick-ogrady
improve refresh rate
7c3d0ea
@patrick-ogrady
remove unnecessary consts
488aaa7
@patrick-ogrady
update public
8cd2ce9
@patrick-ogrady
add error boundary
e1dcc85
@patrick-ogrady
Restore connected (#16)
c48798c 
* restore basic connected

* connection status styling

* fix center
@patrick-ogrady patrick-ogrady marked this pull request as ready for review March 11, 2025 21:34
@patrick-ogrady patrick-ogrady merged commit 0da2b49 into main Mar 11, 2025
4 checks passed
@patrick-ogrady patrick-ogrady deleted the cleanup-react branch March 11, 2025 21:34
@patrick-ogrady patrick-ogrady mentioned this pull request Mar 11, 2025
Closed
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 61.50%. Comparing base (bf0b1ae) to head (c48798c).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
types/src/wasm.rs 0.00% 3 Missing ⚠️ 
@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   61.52%   61.50%   -0.03%     
==========================================
  Files          23       23              
  Lines        3397     3403       +6     
==========================================
+ Hits         2090     2093       +3     
- Misses       1307     1310       +3
Files with missing lines Coverage Δ
chain/src/actors/application/supervisor.rs 95.12% <100.00%> (ø)
types/src/consensus.rs 89.89% <100.00%> (+0.16%) ⬆️
types/src/lib.rs 100.00% <ø> (ø)
types/src/wasm.rs 0.00% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf0b1ae...c48798c. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@patrick-ogrady @codecov-commenter