OLMIS-6566: Updated orderable search query

Activity

FEOLMIS-4010 4

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 16m 1 It looks like it isn't. Moreover, the whole findOrderable...
    Reviewer - Complete 12m    
    Reviewer - Complete 9m 2 Is that still necessary?
    Reviewer - Complete 9m 1 What about changelog? https://review.openlmis.org/static/...
    Total   46m 4  
    #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

    Paulina Buzderewicz

    What about changelog? https://review.openlmis.org/static/ogdo0b/2static/image...

    What about changelog?

    /src/.../repository/OrderableRepositoryIntegrationTest.java Changed
    Open in IDE #permalink
    /src/.../web/OrderableControllerIntegrationTest.java Changed
    Open in IDE #permalink
    /src/main/.../domain/Orderable.java Changed
    Open in IDE #permalink
    /src/.../impl/OrderableRepositoryImpl.java Changed 3
    Open in IDE #permalink
    /src/.../custom/OrderableRepositoryCustom.java Changed
    Open in IDE #permalink
    /src/.../repository/OrderableRepository.java Changed
    /src/.../repository/RepositoryConstants.java Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against