Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-852
  • finished reviewing
Hier kan ik niet veel zinnigs aan reviewen...

Hier kan ik niet veel zinnigs aan reviewen...

[UST3406] Orders die doorgestuurd zijn voor picking, maar daar op status "K" (Kode) staan, worden op...
[UST3406] Orders die doorgestuurd zijn voor picking, maar daar op status "K" (Kode) staan, worden op...
damn

damn

ik denk dat ze hier op hun plaats staan, want bvb voor tekstlijnen ga je nooit een gelinkte toelevering lijn krijgen...

ik denk dat ze hier op hun plaats staan, want bvb voor tekstlijnen ga je nooit een gelinkte toelevering lijn krijgen...

De KOM link IS een toelevering, geen KOM link is dus geen toelevering. Verwarrend is, dat als er KOM in de orderlijn staat, er geen toelevering meer is end dus geen KOM link meer is. De logica van ...

De KOM link IS een toelevering, geen KOM link is dus geen toelevering.
Verwarrend is, dat als er KOM in de orderlijn staat, er geen toelevering meer is end dus geen KOM link meer is.
De logica van de naam is dus in orde

  • More
  • CR-792
  • finished reviewing
GeenKomLink klopt niet 100 % want als er geen komlink is, dan zouden er toeleveringen gemaakt worden na migratie en dat is net wat we niet willen. We bedoelen met deze method dat de orders reeds ve...

GeenKomLink klopt niet 100 % want als er geen komlink is, dan zouden er toeleveringen gemaakt worden na migratie en dat is net wat we niet willen.
We bedoelen met deze method dat de orders reeds verwerkt zijn en het niet nodig is een toelevering aan te maken na migratie.

Ik zou bv opteren voor ToeleveringMakenNaMigratie of MoetVerwerktWordenNaMigratie..

lol. idem.. Klasse wordt momenteel misschien enkel hier gebruikt, maar indien ze toch gebruikt wordt door iemand anders, initvars in de klasse zelf toevoegen.

lol. idem.. Klasse wordt momenteel misschien enkel hier gebruikt, maar indien ze toch gebruikt wordt door iemand anders, initvars in de klasse zelf toevoegen.

Ik heb de zet hier ook toegevoegd, maar zet en geef toeleveringID zitten op de DataMOrder object. Dus te bekijken of het beter is ze samen te zetten in dezelfde klasse. uiteindelijk erft ProductOrd...

Ik heb de zet hier ook toegevoegd, maar zet en geef toeleveringID zitten op de DataMOrder object. Dus te bekijken of het beter is ze samen te zetten in dezelfde klasse. uiteindelijk erft ProductOrderlijn over van Orderlijn.

[UST3269] VHIP504: Export alle orders, met KOM-markering
[UST3269] VHIP504: Export alle orders, met KOM-markering
  • More
  • CR-627
  • finished reviewing
[DEF608] EWRECR-probleem (steeds transferorders?)
[DEF608] EWRECR-probleem (steeds transferorders?)

  • More
  • CR-389
  • finished reviewing
Ik ga ervan uit dat ik de routine MergeOLUNrs^FLOWORD6() niet moet reviewen, hé.

Ik ga ervan uit dat ik de routine MergeOLUNrs^FLOWORD6() niet moet reviewen, hé.

  • More
  • CR-383
  • finished reviewing
Achteraf extra controle dat ^KOD en ^KO1 leeg zijn, zou als "visuele controle" het vertrouwen van de gebruiker nog kunnen verhogen.

Achteraf extra controle dat ^KOD en ^KO1 leeg zijn, zou als "visuele controle" het vertrouwen van de gebruiker nog kunnen verhogen.

Vraag eens aan JoC of Pieter of het toch niet beter is om hier een overkoepelende TransactieScope op te zetten?

Vraag eens aan JoC of Pieter of het toch niet beter is om hier een overkoepelende TransactieScope op te zetten?

Volgens JoC mag je (in code) niet testen op "GetOmgeving" Beter: een specifiek configItem maken (bvb. "MagAlleOrdersVerwijderen") en in de Deploy-klasse de boolean instellen op basis van deze zelfd...

Volgens JoC mag je (in code) niet testen op "GetOmgeving"
Beter:
een specifiek configItem maken (bvb. "MagAlleOrdersVerwijderen") en in de Deploy-klasse de boolean instellen op basis van deze zelfde voorwaarde.
--> in vhDeploy mage je wel testen op GetOmgeving :-P

De klassenaam is een beetje ... raar Of moet ik zeggen : ...reraar http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

De klassenaam is een beetje ... raar
Of moet ik zeggen : ...reraar

[UW396] Order ontsluiten opdat het bij opnieuw sluiten in Admin opnieuw toeleveringen zou maken.
[UW396] Order ontsluiten opdat het bij opnieuw sluiten in Admin opnieuw toeleveringen zou maken.
[UST2567] VHIP504+906: wis orders op Staging en Accept (Admin-kant)
[UST2567] VHIP504+906: wis orders op Staging en Accept (Admin-kant)