Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2634] [rvPVR] Automatische matjessnijder:
[ICT-2634] [rvPVR] Automatische matjessnijder:
De method under test : ToeleveringService.GeefLijnReferentie() kan ook in de Catch{} terecht komen. Dat codepad kan dan best ook ge-unittest worden. In dit geval zal de werkwijze wellicht iets ande...

De method under test : ToeleveringService.GeefLijnReferentie() kan ook in de Catch{} terecht komen.
Dat codepad kan dan best ook ge-unittest worden.
In dit geval zal de werkwijze wellicht iets anders zijn dan bij de gewone "ZetVerwachteExceptie()"
Bij twijfel, vraag gerus.

Bij deze exceptie-omschrijving staat bijzonder weinig info (geen eigenlijk) waardoor de persoon die dit moet troubleshooten sowieso extra tijd zal nodig hebben om uit te pluizen over welke Batch/kl...

Bij deze exceptie-omschrijving staat bijzonder weinig info (geen eigenlijk) waardoor de persoon die dit moet troubleshooten sowieso extra tijd zal nodig hebben om uit te pluizen over welke Batch/klant/palletID/... dit hier gaat. M.a.w. tracht bij een exceptie steeds zoveel mogelijk context te scheppen. En indien dit niet mogelijk is in deze stacklevel, dan zeker checken dat er een Try-Catch op een hoger niveau gebeurt, waar wel meer context aan de exceptie toegevoegd wordt.

Deze opmerking hoort wellicht niet tot de review van deze story, dus feel free om door te geven aan de vorige owner

Over gecombineerde condities (i.e. conditie 1 en conditie 2 hebben een geheel andere context) en waarvan één conditie klantspecifieke filtering is, wil ik je nog een inzicht delen. Uitleg ca. 5 à 1...

Over gecombineerde condities (i.e. conditie 1 en conditie 2 hebben een geheel andere context) en waarvan één conditie klantspecifieke filtering is, wil ik je nog een inzicht delen.
Uitleg ca. 5 à 10 minuten, dus graag effe samenzitten.

[ICT-2954] De Decker - barcodes op etiketten van productie
[ICT-2954] De Decker - barcodes op etiketten van productie
Enkele opmerkingen hierbij: 1) Het etiket zelf is: \\bartender\Bartender\Templates\ProboxLijnReferentieBarcode.btw 2) De DnaCodes in de batch zijn gegroepeerd op DossierCode en LijnReferentie. Vana...

Enkele opmerkingen hierbij:
1) Het etiket zelf is: \\bartender\Bartender\Templates\ProboxLijnReferentieBarcode.btw
2) De DnaCodes in de batch zijn gegroepeerd op DossierCode en LijnReferentie. Vanaf 1 van deze verschillend is wordt een nieuwe 'groep' gemaakt. Echter gekozen voor een sorteeralgoritme omdat dit al bestond en ook wel op een logische manier werkt.
3) De LijnReferentieBepaler is verhuisd naar de ToeleveringService: er waren al 2 exacte dezelfde klasses plus tests hiervoor, een derde zou wat teveel worden. Daarom gekozen om het op een centralere plaats te zetten

De review is een beetje een nest (reverts, code opnieuw toevoegen, ...). Bij onduidelijkheden, bel gerust. Heb hier ook al wat met Wim over gebeld en zei dat het in orde leek (heeft ook een beetje ...

De review is een beetje een nest (reverts, code opnieuw toevoegen, ...). Bij onduidelijkheden, bel gerust. Heb hier ook al wat met Wim over gebeld en zei dat het in orde leek (heeft ook een beetje bijgestuurd). De vhTestFiles zijn allemaal opgeschoven zodat de ProboxOverdoos .txt bestanden daar niet meer bijzitten.

De AansturingIncrementeel voor MVX lijkt in deze review niet te kloppen, maar dat is reeds opgelost in een ander kaartje naar aanleiding van het probleem van vrijdag 18/07. (zie https://vanhoecke.atlassian.net/browse/ICT-2408)

Ik ga dit zeker nog eens testen op accept (zodat niet hetzelfde voorvalt als bij het in productie plaatsen van het samengevoegd PIE) en ik heb al met Wim afgesproken om dit samen in productie te zetten als het zover is.

[ICT-2081] [rvTVE] Afdruk huidig proboxetiket uitschakelen
[ICT-2081] [rvTVE] Afdruk huidig proboxetiket uitschakelen
Ik weet het, het komt van ergens anders, maar toch http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Deze test gaat niet veel doen: ofwel renamen en zien ...

Ik weet het, het komt van ergens anders, maar toch Deze test gaat niet veel doen: ofwel renamen en zien dat ie werkt, ofwel (als hij niet relevant is) verwijderen

[ICT-2105] [rvTVE] Inhoudproboxetiket hernoemen oude verwijzingen
[ICT-2105] [rvTVE] Inhoudproboxetiket hernoemen oude verwijzingen
de %OnNew bovenaan zetten (maakt functioneel niets uit, is conventie. Heeft het voordeel dat je bij openen van klasse direct ziet wat er mogelijks geïnjecteerd wordt)

de %OnNew bovenaan zetten (maakt functioneel niets uit, is conventie. Heeft het voordeel dat je bij openen van klasse direct ziet wat er mogelijks geïnjecteerd wordt)

casing in deze method en die hieronder desnoods nog gelijktrekken

casing in deze method en die hieronder desnoods nog gelijktrekken

Do ipv do

Do ipv do

waarom is deze verwijderd?

waarom is deze verwijderd?

[ICT-2055] [rvTVE] Proboxinhoudetiket samenvoegen in caché
[ICT-2055] [rvTVE] Proboxinhoudetiket samenvoegen in caché
Zoals besproken : *om de svn-history niet te onderbreken, de hernummer-tool gebruiken indien er vhTestFiles bijkomen of wegvallen. *tip 2 : het hernummeren apart committen; dan hoef je dit niet m...

Zoals besproken :

  • om de svn-history niet te onderbreken, de hernummer-tool gebruiken indien er vhTestFiles bijkomen of wegvallen.
  • tip 2 : het hernummeren apart committen; dan hoef je dit niet mee op te nemen in de crucible reviews


Hoeft niets aan te passen voor deze story, maar zeker aan denken bij volgende gelijkaardige situaties.
En ... men zegge het voort!