-
Notifications
You must be signed in to change notification settings - Fork 38
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 19976 int #12676
B 19976 int #12676
Conversation
Bundle StatsHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset (largest 100 files by percent change)
View detailed bundle breakdownAdded
Removed
Bigger
Smaller No assets were smaller Unchanged
|
@@ -48,6 +48,7 @@ const serviceItemCodes = { | |||
MS: 'Move management', | |||
NSTH: 'Nonstandard HHG', | |||
NSTUB: 'Nonstandard UB', | |||
DCRTSA: 'Domestic Standalone Crating', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a double space here. Seems unintentional.
swagger-def/definitions/prime/MTOServiceItemDomesticStandaloneCrating.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functions as expected. Tests are passing. Re-approving from the previous PR for this branch.
Thanks for correcting "ft" to "thousandths of an inch."
I commented on a couple typos that ideally should be fixed but those are minor.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure you update the API change spreadsheet to document your changes.
primemessages.MTOServiceItemModelTypeMTOServiceItemDestSIT: true, | ||
primemessages.MTOServiceItemModelTypeMTOServiceItemShuttle: true, | ||
primemessages.MTOServiceItemModelTypeMTOServiceItemDomesticCrating: true, | ||
primemessages.MTOServiceItemModelTypeMTOServiceItemOriginSIT: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirm if you really need to update V1 for Prime. Any v1 API(s) using it?
19976
Summary
This task adds the ability for the prime to add a 'Standalone Crate' item to a move. Its functionality is nearly identical to a standard crating item, with the caveat that when the TOO goes to approve the item, the highest dollar amount that can be approved is 1000$. The TOO approval is part of a follow-on task and is not completed here.
Verification Steps for the Author
These are to be checked by the author.
Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code
How to test
Prime Sim:
Swagger/ Postman:
Swagger URL: http://primelocal:3000/swagger-ui/prime.html#/mtoServiceItem/createMTOServiceItem
Payload: {"item":{"length":4,"width":4,"height":4},"crate":{"length":5,"width":5,"height":5},"moveTaskOrderID":"958776a4-782f-4657-bd29-0e7f5cf5bedc","mtoShipmentID":"67d0fd70-eba3-4455-83b8-ab658bc8099f","modelType":"MTOServiceItemDomesticStandaloneCrating","reason":"i want to","description":"moped","reServiceCode":"DCRTSA"}
Backend
Database
Any new migrations/schema changes:
Screenshots