Updating commercial metadata

Activity

CR-DOMS-230 37

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 2h 7m 1 Jep
    Reviewer - Complete 1h 15m 36 Jeg savner i hele denne klasse en kort beskrivelse af hvi...
    Total   3h 22m 37  
    #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

    Kåre Fiedler Christiansen

    Jeg savner en README.md fil. Den skal gerne indeholde: *Formået med projekt...

    Jeg savner en README.md fil. Den skal gerne indeholde:

    • Formået med projektet
    • Instruktioner i hvordan det køres/bruges
    • Lidt om design af softwaren, så man kan komme i gang som ny udvikler

    Kåre Fiedler Christiansen

    Tilføj meget gerne en CHANGELOG.md Lige nu kan den selvfølgelig kun sige: 1.0...

    Tilføj meget gerne en CHANGELOG.md
    Lige nu kan den selvfølgelig kun sige:

    1.0 
    
    * Initial version
    

    eller lignende, men den er god at have på plads fra starten

    Kåre Fiedler Christiansen

    Det er altid en god idé at skrive lidt i Objectives i reviewet, for at hjælp ...

    Det er altid en god idé at skrive lidt i Objectives i reviewet, for at hjælp revieweren i gang. I et nyt projekt som dette kan det f.eks. være en læsevejledning i hvor det er klogest at starte med at læse

    /src/main/assembly/assembly.xml Added 1
    Open in IDE #permalink
    /src/.../conf/doms-reklamefixer.properties Added
    Open in IDE #permalink
    /src/main/conf/logback.xml Added
    Open in IDE #permalink
    /src/.../configuration/InitializationFailedException.java Added
    Open in IDE #permalink
    /src/.../configuration/PropertyBasedRegistrarConfiguration.java Added
    Open in IDE #permalink
    /src/main/.../doms/DOMSClient.java Added 2
    Open in IDE #permalink
    /src/.../doms/DOMSCommercialMetadataQuerier.java Added 3
    Open in IDE #permalink
    /src/.../doms/DOMSCommercialMetadataUpdater.java Added 3
    Open in IDE #permalink
    /src/.../reklamefixer/CommercialFixer.java Added 7
    Open in IDE #permalink
    /src/.../reklamefixer/CommercialMetadata.java Added 6
    Open in IDE #permalink
    /src/main/resources/commercial-uuids Added 1
    Open in IDE #permalink
    /src/main/schemas/centralWebservice.wsdl Added 1
    Open in IDE #permalink
    /src/main/schemas/centralWebservice.xsd Added 1
    Open in IDE #permalink
    /src/main/scripts/doms-reklame-metadata.sh Added
    Open in IDE #permalink
    /src/.../java/CommercialFixerIntegrationTest.java Added 1
    Open in IDE #permalink
    /src/.../java/CommercialFixerUnitTest.java Added 2
    Open in IDE #permalink
    /src/.../resources/cinema-metadata-example.xml Added
    Open in IDE #permalink
    /src/.../resources/doms-reklamefixer-test.properties Added
    Open in IDE #permalink
    /src/test/resources/log4j.xml Deleted
    Open in IDE #permalink
    /src/.../resources/tv2-metadata-example.xml Added
    Open in IDE #permalink
    /src/.../resources/updated-cinema-metadata-example.xml Added
    Open in IDE #permalink
    /src/.../resources/updated-tv2-metadata-example.xml Added
    Open in IDE #permalink
    /.gitignore Added
    Open in IDE #permalink
    /pom.xml Added 6
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against