OLMIS-6679: Added a new method to fix Internal Server Errors.

Activity

FEOLMIS-4029 8

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 51m 3 In StockCardRangeSummaryParamsTest.java class there are s...
    Reviewer - Complete 15m 1 Why did we decide to enable searching for stock card summ...
    Reviewer - Complete 19m 4 I see that if a statement is located in StockCardSummarie...
    Reviewer - Complete 7m    
    Total   1h 34m 8  
    #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

    Klaudia Pałkowska

    Why did we decide to enable searching for stock card summaries only by the en...

    Why did we decide to enable searching for stock card summaries only by the end date?

    Paweł Pinker

    To follow the documentation, which says "The date to start including stock da...

    To follow the documentation, which says "The date to start including stock data for the summaries. If not provided summaries will be retrieved starting with the earliest". https://uat.openlmis.org/stockmanagement/docs/

    /src/.../repository/CalculatedStockOnHandRepositoryIntegrationTest.java Changed
    Open in IDE #permalink
    /src/.../repository/CalculatedStockOnHandRepository.java Changed 3
    /src/.../service/StockCardSummariesService.java Changed
    /src/.../service/StockCardSummariesServiceTest.java Changed 3
    /src/.../stockcardrangesummaries/StockCardRangeSummaryParamsTest.java Changed
    Open in IDE #permalink
    /CHANGELOG.md Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time