-
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
4555 [Feature] Update Read-Only View for MMIS changes to Budget and FFP #4605
Conversation
…on open connections
Codecov Report
@@ Coverage Diff @@
## main #4605 +/- ##
===========================================
+ Coverage 77.89% 87.93% +10.04%
===========================================
Files 270 273 +3
Lines 8558 8586 +28
Branches 1749 1760 +11
===========================================
+ Hits 6666 7550 +884
+ Misses 1753 998 -755
+ Partials 139 38 -101
... and 109 files 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.
🚀
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!
…prise-CMCS/eAPD into tforkner/4555-budget-ffp-read-only
Looks good to me! Nice job 😄
|
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.
Changes look 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.
Looks good!
Resolves #4555
Description
Updated Budget and FFP read-only for MMIS. Added default messages if there are no state staff/other state expenses/private contractor costs. Added tests and cleaned up linting issues.
Chromatic Link
https://www.chromatic.com/pullrequest?appId=61d5b948cf6f17003a12bf77&number=4605
Automated test cases written
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 QA has
This pull request can be merged when