Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5031] Re: Template om etiket af te drukken met datum van printen
[ICT-5031] Re: Template om etiket af te drukken met datum van printen
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
ook ergens testen wat er gebeurt als er geen referentie opgehaald kan worden

ook ergens testen wat er gebeurt als er geen referentie opgehaald kan worden

zeker zien dat je de btw file tegelijk in productie zet. qua timing gaat ge daar nog mee moeten opletten denk ik. Best ergens in een pauze, als er zeker geen risico bestaat dat er net op dat moment...

zeker zien dat je de btw file tegelijk in productie zet. qua timing gaat ge daar nog mee moeten opletten denk ik. Best ergens in een pauze, als er zeker geen risico bestaat dat er net op dat moment een ticket wordt afgedrukt.

[ICT-3114] OPO - vermelden van 'klantordernummer' op lade-etiketten en proboxinhoudetiket
[ICT-3114] OPO - vermelden van 'klantordernummer' op lade-etiketten en proboxinhoudetiket
In verband met het statement : "QR-code ... staat op dezelfde plaats op het etiket als de barcode, dus ze mogen niet beide ingevuld zijn" Optie 1 : in de software dit controleren/valideren en een ...

In verband met het statement : "QR-code ... staat op dezelfde plaats op het etiket als de barcode, dus ze mogen niet beide ingevuld zijn"

Optie 1 : in de software dit controleren/valideren en een exceptie smijten wanneer dit zich toch voordoet. De exceptie dan ook in UT opnemen : ZetVerwachteExceptie (je weet wel).
Optie 2 : in de UT tonen dat de software aanvaardt dat beide velden tegelijk ingevuld zijn, dus zonder exceptie te smijten. Hierbij wordt de verantwoordelijkheid dan naar Bartender (in dit geval) doorgeschoven.

Jij hebt hier gekozen voor Optie 2 (bewust of onbewust). Voor mij is dit oké.
Eventueel in die specifieke unit test method toelichten dat dit scenario problemen zal geven in Bartender.

[ICT-3055] VHOSS Etiket QRCode voor De Decker
[ICT-3055] VHOSS Etiket QRCode voor De Decker
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