OLMIS-4197 Change schema from dismissed to dismissTimestamp

Activity

FEOLMIS-2691 3

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 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  
    #permalink

    Objectives

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

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    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