Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Oke, code eens doorlopen en een GetLatest gedaan bij mezelf lokaal zodat bij verdere aanpassing van het TBX Sifonlades ticket ICT-878 de testen ook runnen met deze aangepaste code http://subversion...

Oke, code eens doorlopen en een GetLatest gedaan bij mezelf lokaal zodat bij verdere aanpassing van het TBX Sifonlades ticket ICT-878 de testen ook runnen met deze aangepaste code

@Laura : het verwijderen van stukken code is op zich niet zinvol om te reviewen, maar het kan soms een trigger zijn om andere problemen aan het licht te brengen. In deze fase lijkt me echter onwaar...

@Laura :
het verwijderen van stukken code is op zich niet zinvol om te reviewen, maar het kan soms een trigger zijn om andere problemen aan het licht te brengen.
In deze fase lijkt me echter onwaarschijnlijk dat je mij kunt wijzen op zaken die zullen mislopen door het verwijderen van deze stukken code :-P

ICT-1086: TBX sifon type1 : opkuis oude code+UT
ICT-1086: TBX sifon type1 : opkuis oude code+UT
Is bij de laatste wijzigingen terug F geworden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is bij de laatste wijzigingen terug F geworden

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ? Best toch effe checken bij JohanB.

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ?
Best toch effe checken bij JohanB.

[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
Het ging hier over een bestaande parameter zonder type. ik heb het type toegevoegd.

Het ging hier over een bestaande parameter zonder type. ik heb het type toegevoegd.

  • More
  • CR-228
  • finished reviewing
Moet eigenlijk in /vhUnitTest/.../Service/GeefMogelijkeZendingen/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/GeefMogelijkeZendingen/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/BerekenPalletGegevens/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/BerekenPalletGegevens/Test.cls

hier en daar wat casing aanpassen

hier en daar wat casing aanpassen

$$$Not ipv ' Caché kennende is het ook veiliger om if (()&&()) te doen ipv if () && () (ook hieronder)

$$$Not ipv '
Caché kennende is het ook veiliger om if (()&&()) te doen ipv if () && () (ook hieronder)

Waarom die nieuwe parameter? Er wordt op het eerste zicht niets mee gedaan.

Waarom die nieuwe parameter? Er wordt op het eerste zicht niets mee gedaan.

[UST2424] Vh4: Handels: Transportmogelijkheden doorgeven TBX
[UST2424] Vh4: Handels: Transportmogelijkheden doorgeven TBX