B 21544 prime successful update message int

[KonstanceH]

B-21544

Summary

This PR allows for a success message to flash when the prime updates a shipment's destination address.

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?

How to test

1. Login as the Prime
2. Find a move that you can change the delivery address on. For example an HHG move.
3. Click "Update shipment Delivery Address"
4. Update the address and save
5. Confirm seeing the Success message

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.

Screenshots

[taeJungCaci]

LGTM

[WeatherfordAaron]

lgtm 👽

[ajlusk]

Works good. Looks good.

Couldn't hurt to add a test to help keep the coverage up.
src/pages/MyMove/Agreement.test.jsx:~65 (among others) seems to have a good example to testing the flash message. Lmk if you add one, I'll re-approve quickly.

[KonstanceH]

Works good. Looks good.

Couldn't hurt to add a test to help keep the coverage up. src/pages/MyMove/Agreement.test.jsx:~65 (among others) seems to have a good example to testing the flash message. Lmk if you add one, I'll re-approve quickly.

Ah yea thats a good point! I'll work adding that test in

[ajlusk]

Thanks for adding a test. Sorry it turned out to be a pain.

[KonstanceH]

Thanks for adding a test. Sorry it turned out to be a pain.

It's better to keep up with the test coverage than not. And I definitely need to work on my FE tests so no worries!

[ajlusk]

Those failing client tests say flaky but I think it might instead be a wonky merge. I was in that file recently and those 'Orders type' lines it's failing on need to be /Orders type/ like they are in integrationTesting, but not yet in main.