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

MAIN - B-21022 - Include/Exclude specific offices #13746

Merged
merged 5 commits into from
Sep 23, 2024

Conversation

MInthavongsay
Copy link
Contributor

@MInthavongsay MInthavongsay commented Sep 20, 2024

B-21022

INT PR

Summary

MAIN

This PR addresses issue of certain offices showing and not showing in the close out widget for both customer and office apps when onboarding/editing PPMs. An analysis was done to compare records in the transportation_offices table vs Excel/csv File:
Transportation Offices with Statuses 5 Sep 2024 -Validations.csv with respect to the provides_ppm_closeout field/column. Per Danny M, the Excel/csv file serves as the Gold standard.

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

A migration file was created to update 7 records in the transporation_offices table, updating the provides_ppm_closeout column to match the Excel/csv file. They are the following:

PPPO Fort Belvoir - USA => change FALSE to TRUE
Personal Property Activity HQ (PPA HQ) - USAF => change TRUE to FALSE
PPPO Base Miami - USCG => change FALSE to TRUE
JPPSO - North West (JEAT) - USA => change TRUE to FALSE
PPPO NSWC Panama City Division - USN => change FALSE to TRUE
JPPSO - Mid Atlantic (BGAC) - USA => change TRUE to FALSE
JPPSO - South West (LKNQ) - USN => change TRUE to FALSE

TRUE => Visible in Closeout widget search results (selectable)
FALSE => Not shown/hidden in Closeout widget search results (not selectable)

  1. Login as customer and create and submit a move containing a PPM.
  2. Specifically verify the 7 offices are visible or hidden based on boolean state(provides_ppm_closeout) for the closeout widget. If you want you can do spot checks using anyone one of the offices in the attached Excel file according to the PPM Closeout column.
  3. Login as service counselor and view submitted move PPM. Again verify the 7 offices are either visible or hidden on edit.
  4. Add a new PPM from SC. Do the same verification and spot checks using Excel file as guide.
  5. Are there any other places missed and not specified in the instructions? If so, check that too.

Screenshot 2024-09-17 at 11 41 05 AM

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.

…e to align with Danny M's spreadsheet. Update GHC handler to pass in forPpm parameter to ensure filteriing on ppm_closeout col.
Copy link
Contributor

@r-mettler r-mettler left a comment

Choose a reason for hiding this comment

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

changes same as INT

Copy link
Contributor

@antgmann antgmann left a comment

Choose a reason for hiding this comment

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

Matches!

@r-mettler r-mettler merged commit b4afd6a into main Sep 23, 2024
41 checks passed
@r-mettler r-mettler deleted the B-21022-not-include-JPPSOs branch September 23, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants