Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
True!

True!

Mocht de uiteindelijke layout anders zijn dan wat nu gebruikt wordt ("ProboxInhoud") zou ik dit gewoon in BL.Etiket.ProboxInhoud zetten ipv hier te veranderen.

Mocht de uiteindelijke layout anders zijn dan wat nu gebruikt wordt ("ProboxInhoud") zou ik dit gewoon in BL.Etiket.ProboxInhoud zetten ipv hier te veranderen.

[ICT-4335] [rvTVE] Proboxinhoudetiket - Wijzigen naar formaat 8x11 + vervolgetiket
[ICT-4335] [rvTVE] Proboxinhoudetiket - Wijzigen naar formaat 8x11 + vervolgetiket
[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

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor co...

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor code-completion

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
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