NO-273: extended export to also handle TV channels; before only Radio channels...

Activity

CR-LARM-5 11

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 52m 4 have changed implementation, so process will terminate if...
    Reviewer - Complete 33m 7 (1 defect) We should check here that the new ChaosChannelEntity has ...
    Total   1h 26m 11 (1 defect)  
    #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

    Colin Rosenthal

    Remember to include release notes as a .md file as part of the codebase.

    Remember to include release notes as a .md file as part of the codebase.

    /src/main/config/chaos_channelmapping.xml Added
    Open in IDE #permalink
    /src/main/.../cli/ExportContext.java Changed
    Open in IDE #permalink
    /src/main/.../cli/ExportOptionsParser.java Changed
    Open in IDE #permalink
    /src/.../processors/BtaStatusFetcherDispatcherProcessor.java Changed
    /src/.../processors/DoExportProcessor.java Changed
    /src/.../processors/IsRadioOrTVProgramCheckerProcessor.java Added 2
    Open in IDE #permalink
    /src/.../processors/IsRadioProgramCheckerProcessor.java Deleted
    Open in IDE #permalink
    /src/.../consumer/ConsumerApplication.java Changed
    /src/main/.../consumer/ExportRequestState.java Changed
    /src/main/.../util/ChannelMapper.java Changed 7
    /src/test/.../util/ChannelMapperTest.java Changed
    /src/.../resources/chaos_channelmapping.xml Added 1
    Open in IDE #permalink
    /.gitignore Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time