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 21507 pod poe too validation #14400

Merged
merged 151 commits into from
Jan 8, 2025

Conversation

msaki-caci
Copy link
Contributor

Agility ticket

Summary

Is there anything you would like reviewers to give additional scrutiny?

this article explains more about the approach used.

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
  2. Login as a

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

@msaki-caci msaki-caci changed the base branch from main to MAIN-B-21505 December 13, 2024 19:54
paulstonebraker and others added 21 commits December 16, 2024 18:59
Update API call to pull Port info
Update MTOShipment.yaml to include Port
…on-Submit-move-details-task-order-with-files
Add POE/POD Location to MTOShipment payload
Make methods to pull the port location into the payload
update swagger definitions for POE/POD locations
@robot-mymove
Copy link

robot-mymove commented Jan 6, 2025

Warnings
⚠️

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?

View the frontend file org ADR for more information

Generated by 🚫 dangerJS against d2e43bc

Use 'else if' to improve efficiency
Add poe location to ShipmentDetails.stories.jsx
Hide POE/POD Location UI if there isn't a value
Move port location formatter to formatters.js
Add toTitleCase function to formatters.js
Update test files
Start adding test to mto_shipment_fetcher_test.go
Fix serviceItemPortFSC to link to shipment model and pass DB argument
Update test on ShipmentAddresses.test.jsx
@msaki-caci msaki-caci merged commit d66bebd into integrationTesting Jan 8, 2025
30 of 32 checks passed
@msaki-caci msaki-caci deleted the MAIN-B-21507_POD-POE_TOO_Validation branch January 8, 2025 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.