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 21464 int #2 #14283

Merged
merged 5 commits into from
Nov 26, 2024
Merged

B 21464 int #2 #14283

merged 5 commits into from
Nov 26, 2024

Conversation

CoryKleinjanCACI
Copy link
Contributor

B-21464

Summary

Adding nil check for primeEstimatedWeight and primeActualWeight on recalculating incentive.

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. Access the MilMove application
  2. Login as a Customer
  3. Create a Move with an HHG and an NTS-Release with non california zips (this is done so that all service item types can be tested)
  4. Login as a Service Counselor
  5. Submit the Move details for the Move you created
  6. Login as a TOO
  7. Approve the shipments for the move you created
  8. Login to the Prime simulator
  9. Select the move you created
  10. Update the HHG and NTS shipments with Pickup/Dropoff dates and weights
  11. Select the "Update Destination Address" button for both the NTS and HHG, and request the destination address updated to California Zip 90210
  12. Login as a TOO
  13. Select to edit the HHG shipment
  14. Select the option to review the destination address change
  15. Approve the change
  16. Select to edit the NTS shipment
  17. Select the option to review the destination address change
  18. Approve the change
  19. Navigate to the Move Task Order page and verify the the service items have estimated prices
  20. Login to the Prime simulator
  21. Select the move you created
  22. Select the "Update Destination Address" button for both the NTS and HHG, and request the destination address updated to Kentucky Zip 42420
  23. Login as a TOO
  24. Select to edit the HHG shipment
  25. Select the option to review the destination address change
  26. Approve the change
  27. Select to edit the NTS shipment
  28. Select the option to review the destination address change
  29. Approve the change
  30. Navigate to the Move Task Order page and verify the the service items have different estimated prices

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

@CoryKleinjanCACI CoryKleinjanCACI added Mountain Movers Movin' Mountains 1 Sprint at a time Scrummy Bears Scrum Team H INTEGRATION Slated for Integration Testing and removed Mountain Movers Movin' Mountains 1 Sprint at a time labels Nov 26, 2024
@CoryKleinjanCACI CoryKleinjanCACI self-assigned this Nov 26, 2024
Copy link
Contributor

@deandreJones deandreJones left a comment

Choose a reason for hiding this comment

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

simple enough

@CoryKleinjanCACI CoryKleinjanCACI merged commit 712f409 into integrationTesting Nov 26, 2024
30 checks passed
@CoryKleinjanCACI CoryKleinjanCACI deleted the B-21464-INT branch November 26, 2024 20:11
@CoryKleinjanCACI CoryKleinjanCACI restored the B-21464-INT branch December 3, 2024 04:56
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.

3 participants