OLMIS-4197 Change schema from dismissed to dismissTimestamp


FEOLMIS-2691 3

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 4m 1 Probably; it's probably habit from the old Hibernate gene...
    Reviewer - Complete 13m 2 Left a curiosity question. LGTM, thanks!
    Total   16m 3  


    • Review dismissed boolean now dismissTimestamp timestamp with time zone. Change in domain object, DTO, schema migration, tests, demo data.

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    Josh Zamor

    Left a curiosity question. LGTM, thanks!

    Left a curiosity question. LGTM, thanks!

    /demo-data/cce.cce_alerts.json Changed
    /src/.../repository/AlertRepositoryIntegrationTest.java Changed
    /src/.../web/AlertControllerIntegrationTest.java Changed
    /src/main/java/.../cce/domain/Alert.java Changed 2
    /src/main/java/.../cce/dto/AlertDto.java Changed
    /src/.../migration/20180216055219230__create_cce_alerts.sql Changed
    /src/test/java/.../cce/domain/AlertTest.java Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against