Skip to content
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

Merged
merged 34 commits into from
Mar 27, 2023

Conversation

thetif
Copy link
Contributor

@thetif thetif commented Mar 14, 2023

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

Given When Then Type (jest, tap, cypress)
an APD there is no other funding it doesn't have other funding in the list jest
an APD there is other funding other funding appears in the list jest
a HITECH APD displaying Budget and FFP for Activity 1 Key State Personnel are listed with State Staff jest
a MMIS APD displaying Budget and FFP doesn't list Key State Personnel jest
an APD there is no State Staff/Other State Expenses/Private Contract it display the default message jest

Steps to manually verify this change

  1. open a HITECH APD
  2. navigate to Export and Submit
  3. click on Continue to review
  4. scroll down to the Budget and FFP section of Activity 1
  5. verify that it displays like it did before

This pull request is ready to code review when

  • Automated tests are updated (and all tests are passing)
  • New automated test cases are documented above
  • Chromatic link added above
  • Pull request has been labeled, if applicable with feature, content, bug,
    tests, refactor
  • The experience passes a basic manual accessibility audit (keyboard nav,
    screenreader, text scaling) OR an exemption is documented

This pull request is ready to test when

  • Code has been reviewed by someone other than the original author

This pull request is ready to review when QA has

  • Verified the functionality related to the change
  • Verified that the change works with Narrator on Windows
  • Verified that the change works with VoiceOver on Mac
  • Verified all updated pages with the WAVE tool
  • Verified tab and keyboard navigation functionality

This pull request can be merged when

  • Design has approved the experience
  • Product has approved the experience

@thetif thetif added the feature PR label for release log label Mar 14, 2023
@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Merging #4605 (2b344bb) into main (a0a75f6) will increase coverage by 10.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             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     
Flag Coverage Δ
api ∅ <ø> (?)
common 99.33% <ø> (ø)
web 87.30% <100.00%> (+10.59%) ⬆️
Impacted Files Coverage Δ
...eb/src/pages/apd/activities/ffp/CostAllocateFFP.js 97.56% <ø> (+4.70%) ⬆️
...d/activities/cost-allocation/CostAllocationRows.js 100.00% <100.00%> (+12.76%) ⬆️
...es/apd/proposed-budget/SummaryActivityBreakdown.js 100.00% <100.00%> (ø)

... and 109 files with indirect coverage changes


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 a0a75f6...2b344bb. Read the comment docs.

@cms-eapd-bot
Copy link

cms-eapd-bot commented Mar 15, 2023

See this pull request in action: https://ec2-34-193-194-48.compute-1.amazonaws.com

2b344bb

@thetif thetif requested review from tbolt and Sun-Mountain March 16, 2023 20:42
@thetif thetif requested review from amyd11 and removed request for Sun-Mountain March 16, 2023 20:42
@thetif thetif requested a review from Sun-Mountain March 17, 2023 17:47
Copy link
Contributor

@Sun-Mountain Sun-Mountain left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Contributor

@amyd11 amyd11 left a comment

Choose a reason for hiding this comment

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

Looks good!

@mirano-darren mirano-darren self-requested a review March 21, 2023 16:21
@thetif thetif changed the title Tforkner/4555 budget ffp read only 4555 [Feature] Update Read-Only View for MMIS changes to Budget and FFP Mar 22, 2023
@mirano-darren
Copy link
Contributor

Looks good to me! Nice job 😄

Given When Then Covered
a HITECH APD On the export view Budget and FFP section should be the same and not have any MMIS updates
a MMIS APD On the export view (use Chromatic if page crashes) Budget and FFP section should match the data in the Budget and FFP page
a MMIS APD Budget and FFP data is changed (added costs/deleted costs/edited costs) Budget and FFP section should match the changed data 🟨 Export view not available to test
a MMIS APD Budget and FFP data is blank Budget and FFP section should match the empty state

Copy link

@SGilliamA1M SGilliamA1M left a comment

Choose a reason for hiding this comment

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

Changes look good.

Copy link
Contributor

@jeromeleecms jeromeleecms left a comment

Choose a reason for hiding this comment

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

Looks good!

@thetif thetif merged commit aeb5d2b into main Mar 27, 2023
@thetif thetif deleted the tforkner/4555-budget-ffp-read-only branch March 27, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature PR label for release log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Update Read-Only View for MMIS changes to Budget and FFP
9 participants