OLMIS-3933, Allow controller to update an Orderable through PUT


FEOLMIS-3690 9

Keyboard shortcuts  
  • Summarize the review outcomes (optional)


    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 12m 4 Hmm, it should either ignore or replace associations from...
    Reviewer - Complete 41m 5 Josh Zamor and Chongsun Ahn FYI, I think resolving may re...
    Reviewer - Complete 4m    
    Total   57m 9  


    Adding to the controller, this isn't done done so see the issue. e.g. this doesn't address version increments, RAML updates, logging a warning on use, etc. All I could get in before rushing out the door!

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    There are no general comments on this review.
    /src/.../web/OrderableControllerIntegrationTest.java Changed
    /src/main/.../domain/Orderable.java Changed
    /src/.../messagekeys/OrderableMessageKeys.java Changed
    /src/main/.../referencedata/util/UuidUtil.java Changed
    Open in IDE #permalink
    /src/main/.../validate/OrderableValidator.java Changed
    Open in IDE #permalink
    /src/main/.../web/OrderableController.java Changed 9
    /src/main/resources/messages_en.properties Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against