OLMIS-7219: Updated POST /order and /orders/batch to return existing orders if...

Activity

FEOLMIS-4326 6

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 8m 1 NullPointer is thrown in that case.
    Reviewer - Complete 9m 5 Let's not change behavior for this endpoint
    Reviewer - Complete 0m    
    Total   18m 6  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /.env Changed
    Open in IDE #permalink
    /deployment/perftest_env/.env Changed
    /deployment/test_env/.env Changed
    Open in IDE #permalink
    /deployment/uat_env/.env Changed
    /src/.../repository/OrderRepositoryIntegrationTest.java Changed
    /src/.../web/OrderControllerIntegrationTest.java Changed
    /src/main/.../i18n/MessageKeys.java Changed
    Open in IDE #permalink
    /src/main/.../repository/OrderRepository.java Changed
    Open in IDE #permalink
    /src/main/.../web/OrderController.java Changed 3
    /src/main/resources/api-definition.yaml Changed 2
    Open in IDE #permalink
    /src/main/resources/messages_en.properties Changed
    Open in IDE #permalink
    /CHANGELOG.md Changed 1
    /gradle.properties Changed
    /.env Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time