Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wanneer meer dan 90% van de klassen verwijderd worden, dan valt er nog weinig te reviewen. De blijvende implementaties uiteraard, die zijn "Review O.K." Indien een klasse onterecht verwijderd zou z...

Wanneer meer dan 90% van de klassen verwijderd worden, dan valt er nog weinig te reviewen. De blijvende implementaties uiteraard, die zijn "Review O.K."
Indien een klasse onterecht verwijderd zou zijn, zal ik als reviewer hier helaas niet kunnen opmerken.
Er zijn wel nog enkele deploy-klassen die verwijzen naar de package "APPS.common.KlantKost". Deze kan je best ook nog opkuisen.

[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik.
[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik.
cosmetica: spaties rond = (ook in testje hieronder)

cosmetica: spaties rond = (ook in testje hieronder)

[UW714] GLS: Nieuwe range voor Parcel instellen
[UW714] GLS: Nieuwe range voor Parcel instellen
[VRB] MAG: Transport: GLS-integratie: duidelijkere foutmelding naar gebruiker/ICT bij E001: a mandatory...
[VRB] MAG: Transport: GLS-integratie: duidelijkere foutmelding naar gebruiker/ICT bij E001: a mandatory...
[UW479] Types uitklaren GLS - verschillende types door elkaar + 1 onbestaande klasse.
[UW479] Types uitklaren GLS - verschillende types door elkaar + 1 onbestaande klasse.
Ineens #class(APPS.TRANSP.TransportAPI).IsGLS(Obj.Transporteur) gebruiken. Dat is op andere plaatsen ook zo aangepast

Ineens #class(APPS.TRANSP.TransportAPI).IsGLS(Obj.Transporteur) gebruiken. Dat is op andere plaatsen ook zo aangepast

[UW470] GLS - nieuwe depot, nieuwe klantnr, nieuwe range.
[UW470] GLS - nieuwe depot, nieuwe klantnr, nieuwe range.
  • More
  • CR-983
  • finished reviewing
Om de testen te goei te hebben moet er heel wat opgekuist worden in code van Jokke. Ik heb het kaartje voorlopig al op te reviewen gezet omdat het bij mij lokaal al werkt. Voor mijn part wachten we...

Om de testen te goei te hebben moet er heel wat opgekuist worden in code van Jokke. Ik heb het kaartje voorlopig al op te reviewen gezet omdat het bij mij lokaal al werkt. Voor mijn part wachten we met doormergen tot de testen klaar zijn, maar dan moet wel manueel de Range-global op 2 gezet worden als de Parcelrange >98%

[ICT353] GLS Freight Range
[ICT353] GLS Freight Range
  • More
  • CR-248
  • finished reviewing
Niet echt nodig daar het GLS eigenschappen zijn die enkel maar voor bepaalde testen gebruikte worden. De vhUnitTest prefix zegt al genoeg dat het niet over een echten gaat.

Niet echt nodig daar het GLS eigenschappen zijn die enkel maar voor bepaalde testen gebruikte worden. De vhUnitTest prefix zegt al genoeg dat het niet over een echten gaat.

Deze class zou beter ge-renamed worden naar Fake of Mock (???) Dat weet gij beter dan ik http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Deze class zou beter ge-renamed worden naar Fake of Mock (???)
Dat weet gij beter dan ik

variabale naam "HuidigeRange" lijkt me nogal verwarrend. Probeer ietske met ...Id ?

variabale naam "HuidigeRange" lijkt me nogal verwarrend. Probeer ietske met ...Id ?