-
Notifications
You must be signed in to change notification settings - Fork 30
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.
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
Tbolt/4506 Updates to proposed budget and executive summary #4583
Tbolt/4506 Updates to proposed budget and executive summary #4583
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4583 +/- ##
==========================================
+ Coverage 94.30% 94.31% +0.01%
==========================================
Files 273 273
Lines 8660 8694 +34
Branches 1721 1746 +25
==========================================
+ Hits 8167 8200 +33
- Misses 469 470 +1
Partials 24 24
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
See this pull request in action: https://ec2-34-193-194-48.compute-1.amazonaws.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job!
Looks good to me! Note: Executive summary couldn't be checked (page still crashes) so I checked them in Chromatic
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice - this looks like the Figma. Still other tickets to revise the tables, but this covers the scope of the ticket!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work on the table updates @tbolt!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, Well done!
Resolves #4506
Description
Updates to table designs
Chromatic approval
Significant changes or possible side effects
Automated test cases written
No new tests needed for these design changes. Tests were updated where needed.
Steps to manually verify this change
This pull request is ready to code review when
tests, refactor
screenreader, text scaling) OR an exemption is documented
This pull request is ready to test when
This pull request is ready to review when the QA has
This pull request can be merged when