Updating commercial metadata

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 Changed 6
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create issue

X
Assign To Me

Log time against