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 21680 int #14242

Merged
merged 28 commits into from
Nov 26, 2024
Merged

B 21680 int #14242

merged 28 commits into from
Nov 26, 2024

Conversation

pambecker
Copy link
Contributor

@pambecker pambecker commented Nov 20, 2024

Agility ticket

Summary

When a customer creates a new move, the Counseling Office field does not immediately load. It takes a few seconds before the data is available to select. At some times, it get a timeout error and never loads. This is preventing any new moves to be created by the customer on Integration.
Also, there was an issue when you change Current Duty Location, it wasn't resetting the Counseling Office dropdown value, and then allowing the customer to submit with a blank Counseling office. This has been fixed as well.

How to test

  1. Login as a Milmove customer and create a new move.
  2. After picking a New Duty Location from the dropdown, go to choose a Counseling office.
  3. Verify that it loads relatively quick, 2-3 seconds roughly.
  4. Go back and change the New Duty Location selection.
  5. Before picking a new Counseling Office, verify that Counseling Office is showing the red Required label and that Next is not enabled.
  6. Add a Counseling Office selection and save.

@pambecker pambecker self-assigned this Nov 20, 2024
@pambecker pambecker added G-Unit Scrum Team G INTEGRATION Slated for Integration Testing labels Nov 20, 2024
@pambecker pambecker marked this pull request as ready for review November 20, 2024 15:50
@pambecker pambecker requested a review from a team November 20, 2024 15:58
@pambecker pambecker marked this pull request as draft November 20, 2024 22:55
@pambecker pambecker marked this pull request as ready for review November 22, 2024 19:44
Copy link
Contributor

@loganwc loganwc left a comment

Choose a reason for hiding this comment

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

Looks great

@pambecker pambecker merged commit 4711e21 into integrationTesting Nov 26, 2024
30 checks passed
@pambecker pambecker deleted the B-21680-INT branch November 26, 2024 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
G-Unit Scrum Team G INTEGRATION Slated for Integration Testing
Development

Successfully merging this pull request may close these issues.

5 participants