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

B 20290 int add amount reimbursement #13166

Merged
merged 20 commits into from
Jul 9, 2024
Merged

Conversation

r-mettler
Copy link
Contributor

B-20290

Summary

This story adds Actual SIT Reimbursement field to the Review Documents page for storage moving expenses. The value should be the smaller of the Government SIT Cost and the Amount Requested. The Actual SIT Reimbursement amount will adjust as SIT Cost and Amount Requested are updated. The Actual SIT Reimbursement amount is saved into a new DB column in the moving_expenses table and will be loaded into the SSW for shipments that have a storage type moving expense. The SSW was also updated to use the Actual SIT Reimbursement amount for PPM shipments with storage moving expense in the Member-paid SIT field.

image

Verification Steps for the Author

These are to be checked by the author.

  • Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
  • Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers

These are to be checked by a reviewer.

  • Has the branch been pulled in and checked out?
  • Have the BL acceptance criteria been met for this change?
  • Was the CircleCI build successful?
  • Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

How to test

  1. Login as a Customer
  2. Create a PPM shipment and make sure you enter Yes when asked if you plan to use storage
  3. Submit the shipment
  4. Login as a Service Counselor
  5. Approve the shipment
  6. Login as the customer
  7. Upload PPM documents and fill out the first few pages
  8. Make sure you add a Storage type moving expense to the shipment
  9. Submit the documents
  10. Login as the Service Counselor
  11. Go to the PPM Closeout Queue select the shipment
  12. Click on the Review Documents button
  13. You should see the Actual SIT Reimbursement which is the lower of the Government SIT Cost and Amount Requested
  14. Adjust the location or other SIT items so the cost updates and also adjust Amount Requested verify Actual SIT Reimbursement is the lower of the two amounts
  15. Accept and approve the documents
  16. Click the download link for the Payment Packet
  17. Look at the Member-paid SIT field and verify the amount is the same as was shown on the Actual SIT Reimbursement displayed on the Review Documents page

Frontend

  • There are no aXe warnings for UI.
  • This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
  • There are no new console errors in the browser devtools.
  • There are no new console errors in the test output.
  • If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
  • This change meets the standards for Section 508 compliance.

Backend

Database

Any new migrations/schema changes:

  • Follows our guidelines for Zero-Downtime Deploys.
  • Have been communicated to #g-database.
  • Secure migrations have been tested following the instructions in our docs.

Screenshots

@r-mettler r-mettler self-assigned this Jun 28, 2024
@r-mettler r-mettler removed their assignment Jun 28, 2024
@r-mettler r-mettler added Scrummy Bears Scrum Team H INTEGRATION Slated for Integration Testing labels Jun 28, 2024
@antgmann antgmann marked this pull request as ready for review July 1, 2024 19:08
@antgmann antgmann requested a review from a team as a code owner July 1, 2024 19:08
Copy link
Contributor

@CoryKleinjanCACI CoryKleinjanCACI left a comment

Choose a reason for hiding this comment

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

Code looks good and test steps pass.

dependabot bot and others added 5 commits July 8, 2024 11:13
Bumps [github.com/gorilla/csrf](https://github.com/gorilla/csrf) from 1.7.1 to 1.7.2.
- [Release notes](https://github.com/gorilla/csrf/releases)
- [Commits](gorilla/csrf@v1.7.1...v1.7.2)

---
updated-dependencies:
- dependency-name: github.com/gorilla/csrf
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
…com/gorilla/csrf-1.7.2

Bump github.com/gorilla/csrf from 1.7.1 to 1.7.2
Bumps [github.com/jarcoal/httpmock](https://github.com/jarcoal/httpmock) from 1.3.0 to 1.3.1.
- [Release notes](https://github.com/jarcoal/httpmock/releases)
- [Commits](jarcoal/httpmock@v1.3.0...v1.3.1)

---
updated-dependencies:
- dependency-name: github.com/jarcoal/httpmock
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
…com/jarcoal/httpmock-1.3.1

Bump github.com/jarcoal/httpmock from 1.3.0 to 1.3.1
Copy link
Contributor

@ajlusk ajlusk left a comment

Choose a reason for hiding this comment

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

Functions as described. Tests passing. No notes from code review.
Screenshot 2024-07-09 at 10 40 13 AM
Screenshot 2024-07-09 at 10 41 30 AM

@r-mettler r-mettler merged commit fae5b2f into integrationTesting Jul 9, 2024
32 checks passed
@r-mettler r-mettler deleted the B-20290-INT branch July 9, 2024 16:40
@r-mettler r-mettler mentioned this pull request Jul 16, 2024
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTEGRATION Slated for Integration Testing Scrummy Bears Scrum Team H
Development

Successfully merging this pull request may close these issues.

5 participants