OLMIS-6361 Added UI for displaying system notifications on Home page

Activity

FEOLMIS-3846 1

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 11m    
    Reviewer - Complete 4m    
    Reviewer - Complete 12m 1 Shouldn't we add assert for author field to check whether...
    Reviewer - 0% reviewed      
    Total   27m 1  
    #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.
    /src/.../web/SystemNotificationControllerIntegrationTest.java Changed 1
    /src/main/.../dto/SystemNotificationDto.java Changed
    /src/main/.../dto/UserObjectReferenceDto.java Added
    Open in IDE #permalink
    /src/.../web/SystemNotificationController.java Changed
    /src/.../web/SystemNotificationSearchParams.java Changed
    Open in IDE #permalink
    /src/.../schemas/systemNotificationDto.json Changed
    /src/main/.../schemas/userObjectReference.json Added
    Open in IDE #permalink
    /src/main/resources/api-definition.yaml Changed
    /src/.../dto/UserObjectReferenceDtoTest.java Added
    Open in IDE #permalink
    /src/.../validate/SystemNotificationValidatorTest.java Changed
    Open in IDE #permalink
    /src/.../web/SystemNotificationSearchParamsTest.java Changed
    Open in IDE #permalink
    /CHANGELOG.md Changed
    /src/openlmis-header/header.html Changed
    Open in IDE #permalink
    /src/openlmis-header/header.scss Changed
    Open in IDE #permalink
    /src/openlmis-header/openlmis-header.module.js
    Open in IDE #permalink
    /src/openlmis-header/user.html
    Open in IDE #permalink
    /src/openlmis-home/home-system-notifications.controller.js Added
    Open in IDE #permalink
    /src/openlmis-home/home-system-notifications.controller.spec.js Added
    Open in IDE #permalink
    /src/openlmis-home/home.html Changed
    /src/openlmis-home/home.routes.js Changed
    /src/openlmis-home/home.routes.spec.js Added
    /src/openlmis-home/home.scss
    Open in IDE #permalink
    /src/openlmis-home/messages_en.json Added
    Open in IDE #permalink
    /src/openlmis-home/openlmis-home.module.js
    Open in IDE #permalink
    /src/openlmis-system-notifications-indicator/messages_en.json Added
    Open in IDE #permalink
    /src/openlmis-system-notifications-indicator/openlmis-system-notifications-indicator.controller.js Added
    /src/openlmis-system-notifications-indicator/openlmis-system-notifications-indicator.controller.spec.js Added
    /src/openlmis-system-notifications-indicator/system-notifications-indicator.component.js Added
    Open in IDE #permalink
    /src/openlmis-system-notifications-indicator/system-notifications-indicator.html Added
    Open in IDE #permalink
    /src/openlmis-system-notifications-indicator/system-notifications-indicator.module.js Added
    Open in IDE #permalink
    /CHANGELOG.md Changed
    /src/openlmis-message/message.component.js Added
    Open in IDE #permalink
    /src/openlmis-message/message.html Added
    Open in IDE #permalink
    /src/openlmis-message/message.scss Added
    /src/openlmis-message/openlmis-message.module.js Added
    Open in IDE #permalink
    /CHANGELOG.md Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against