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

B21575-INT-add order types #14091

Closed
wants to merge 72 commits into from

Conversation

JamesHawks224
Copy link
Contributor

@JamesHawks224 JamesHawks224 commented Nov 1, 2024

Agility ticket

B21575

Summary

For the customer, 2 new order types were added: Student Travel, Early Return of Dependents. If either of these were selected, then "Has Dependents" would default to "yes" and be disabled so it can't be changed. If another order type besides these 2 were selected, "Has Dependents" would be re-enabled and would be reset to nothing selected.

How to test

Scenario 1: Creating a move

  • login as a customer
  • create a move
  • at the move orders page ("tell us about your move orders")
    • set initial values all fields:
      • order type: Permanent Change of Station
      • remaining fields: any value is fine
  • verification
    • verify: order type, changing existing type to another existing type
      • change order type to local move
      • verify no fields are changed
    • verify: order type, changing existing type to new type (student travel)
      • change order type to student travel
      • verify "are dependents included" defaults to "yes"
      • verify "are dependents included" is disabled
    • verify: order type, changing new type (student travel) to existing type
      • change order type to permanent change of station
      • verify "are dependents included" defaults to no selection
      • verify "are dependents included" is enabled
    • verify: order type, changing existing type to new type (early return of dependents)
      • change order type to student travel
      • verify "are dependents included" defaults to "yes"
      • verify "are dependents included" is disabled
    • verify: order type, changing new type (early return of dependents) to existing type
      • change order type to permanent change of station
      • verify "are dependents included" defaults to no selection
      • verify "are dependents included" is enabled
    • verify: order type, changing new type (early return of dependents) to to new type (student travel)
      • change order type to early return of dependents
      • verify "are dependents included" defaults to "yes"
      • verify "are dependents included" is disabled
      • change order type to student travel
      • verify "are dependents included" defaults to "yes"
      • verify "are dependents included" is disabled
  • finish creating the move w/a ppm shipment.

Scenario 2: edit existing orders

  • login as a customer
  • use the move just created in scenario 1
  • click on "review your request"
  • edit orders
  • on the "edit orders" page, set the initial values the same as in scenario 1
  • perform verification the same verification as in scenario 1.
  • save the page

ANDI Screenshots

Creating a move:
image

Editing existing orders:
image

Testing Screenshots

N/A

@JamesHawks224 JamesHawks224 added ByteSize M&Ms Team ByteSized M&Ms INTEGRATION Slated for Integration Testing labels Nov 1, 2024
@JamesHawks224 JamesHawks224 force-pushed the B21575-INT-Add_order_types branch from daf2e0c to d253dfb Compare November 8, 2024 17:37
Copy link
Contributor

mergify bot commented Nov 8, 2024

⚠️ The sha of the head commit of this PR conflicts with #14046. Mergify cannot evaluate rules on this PR. ⚠️

@JamesHawks224 JamesHawks224 requested a review from a team as a code owner November 11, 2024 20:34
Copy link
Contributor

mergify bot commented Nov 11, 2024

⚠️ The sha of the head commit of this PR conflicts with #14046. Mergify cannot evaluate rules on this PR. ⚠️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ByteSize M&Ms Team ByteSized M&Ms INTEGRATION Slated for Integration Testing
Development

Successfully merging this pull request may close these issues.

10 participants