-
Notifications
You must be signed in to change notification settings - Fork 0
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
Dt 786 #71
Dt 786 #71
Conversation
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.
I am missing scenario based checks on all payloads.
As an example, the carrier is not preserving the active reefer settings, the carrier must fail the scenario. If the shipper by mistake omits the scenario details in a UC3 (PUT) then they should also fail the scenario.
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.
Added some more remarks.
In addition, I do not see a fix for the remark I made inhttps://github.com//pull/71#pullrequestreview-1905776565
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
...c/main/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-reefer-temp-change.json
Outdated
Show resolved
Hide resolved
booking/src/main/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-dg.json
Outdated
Show resolved
Hide resolved
booking/src/main/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-reefer.json
Outdated
Show resolved
Hide resolved
...ng/src/main/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-regular-2re1c.json
Outdated
Show resolved
Hide resolved
...src/main/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-regular-cho-dest.json
Outdated
Show resolved
Hide resolved
...src/main/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-regular-cho-orig.json
Outdated
Show resolved
Hide resolved
...ources/standards/booking/messages/booking-api-2.0.0-Beta-1-regular-non-operating-reefer.json
Outdated
Show resolved
Hide resolved
...ain/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-regular-shipper-owned.json
Outdated
Show resolved
Hide resolved
booking/src/main/resources/standards/booking/messages/booking-api-2.0.0-Beta-1-regular.json
Outdated
Show resolved
Hide resolved
I understood now what you actually mean.. I will put these checks across different use cases. |
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
...ng/src/main/java/org/dcsa/conformance/standards/booking/action/Shipper_GetBookingAction.java
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/dcsa/conformance/core/check/JsonAttribute.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
...ng/src/main/java/org/dcsa/conformance/standards/booking/model/PersistableCarrierBooking.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
...ng/src/main/java/org/dcsa/conformance/standards/booking/model/PersistableCarrierBooking.java
Show resolved
Hide resolved
core/src/main/java/org/dcsa/conformance/core/check/JsonAttribute.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
booking/src/main/java/org/dcsa/conformance/standards/booking/checks/BookingChecks.java
Outdated
Show resolved
Hide resolved
...ng/src/main/java/org/dcsa/conformance/standards/booking/model/PersistableCarrierBooking.java
Show resolved
Hide resolved
My issues have been fixed (also the |
No description provided.