Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5184] [rvJBA] Verplaatsen van batch in de pitching excel
[ICT-5184] [rvJBA] Verplaatsen van batch in de pitching excel
Vrij veel code in 1 method, ik zou die wat meer opsplitsen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif. Ik zie ook geen UT's, dat kan op die manier ...

Vrij veel code in 1 method, ik zou die wat meer opsplitsen . Ik zie ook geen UT's, dat kan op die manier wel opgelost worden

Commentaar mag weg

Commentaar mag weg

Is deze implementatie niet heel gelijkaardig aan die voor MVX? Lijkt mij dat deze in de common klasse kan http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is deze implementatie niet heel gelijkaardig aan die voor MVX? Lijkt mij dat deze in de common klasse kan

[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode
[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode
[ICT-3884] VHLOG MES M1 - eerste controles op gegenereerd json-bestand:
[ICT-3884] VHLOG MES M1 - eerste controles op gegenereerd json-bestand:
[ICT-3794] VHLOG MES M1 - (3) nieuwe aansturingactie 'batch naar OBJT':
[ICT-3794] VHLOG MES M1 - (3) nieuwe aansturingactie 'batch naar OBJT':
Hier staat nog een regel commentaar dat waarschijnlijk weg mag

Hier staat nog een regel commentaar dat waarschijnlijk weg mag

Geen idee of dit van u is, maar hier staat een schrijffout http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Geen idee of dit van u is, maar hier staat een schrijffout

[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
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
Aparte commits omdat ze niet allemaal samen konden gemerged worden.

Aparte commits omdat ze niet allemaal samen konden gemerged worden.