Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-281
  • finished reviewing
?? Iets te ijverig

?? Iets te ijverig

?? Iets te ijverig

?? Iets te ijverig

[ICT260] Voorbereiding interfaces
[ICT260] Voorbereiding interfaces
  • More
  • CR-249
  • finished reviewing
Bitter weinig testjes rond het nieuwe en bestaande gedrag van de SalesInvoiceConverter, behalve dan die CorrigeerAfrondingsVerschil...

Bitter weinig testjes rond het nieuwe en bestaande gedrag van de SalesInvoiceConverter, behalve dan die CorrigeerAfrondingsVerschil...

[UST2057] - Interface verkoopfactuur - wijziging ivm BTW-artikelgroep
[UST2057] - Interface verkoopfactuur - wijziging ivm BTW-artikelgroep
  • More
  • CR-246
  • finished reviewing
Hebben we hier niet wat veel duplicate code , Misschien beter om ergens een private method te maken : alla AssertDatErGeenMailVerstuurdWordt

Hebben we hier niet wat veel duplicate code ,

Misschien beter om ergens een private method te maken : alla AssertDatErGeenMailVerstuurdWordt

[ICT255] Beperking emails AX
[ICT255] Beperking emails AX
  • More
  • CR-190
  • finished reviewing
  • More
  • CR-191
  • resumed reviewing
Private?

Private?

Is het niet nodig om het gedrag van deze method te verduidelijken dmv een paar testjes? Of door op te nemen in reeds bestaande tests? En ja, ik weet dat het een lokale (private) method is en dat ge...

Is het niet nodig om het gedrag van deze method te verduidelijken dmv een paar testjes? Of door op te nemen in reeds bestaande tests? En ja, ik weet dat het een lokale (private) method is en dat ge unittests schrijft en geen methodtests en jadajadajada, maar voor een leek zoals ik is het niet onmiddellijk duidelijk hoe het gewijzigde gedrag het probleem zoals omschreven in de titel oplost. VoilĂ .

Let the rebuttal begin...

private?

private?

  • More
  • CR-169
  • finished reviewing
[DEF571] - AX ContactInfo problemen met combinatie delete en update
[DEF571] - AX ContactInfo problemen met combinatie delete en update
[DEF572] - Klanten die op regio 98 - 99 geplaatst worden gaan niet goed naar AX
[DEF572] - Klanten die op regio 98 - 99 geplaatst worden gaan niet goed naar AX
[UST2301] Klant-sync errors in ICT_meldingen
[UST2301] Klant-sync errors in ICT_meldingen
Het is kort voor "Als de kredietlimiet 1 is (zo zichtbaar in Admin), dan moet er geen 1000 naar AX worden gestuurd, in andere gevallen vermenigvuldigen we wel met 1000 (en zetten wer er 20 nullen a...

Het is kort voor "Als de kredietlimiet 1 is (zo zichtbaar in Admin), dan moet er geen 1000 naar AX worden gestuurd, in andere gevallen vermenigvuldigen we wel met 1000 (en zetten wer er 20 nullen achter de komma bij)", maar dat vond ik te lang als omschrijving. (en caché vond dat ook)

  • More
  • CR-48
  • finished reviewing
Test benaming moet toch eerder zijn als kredietlimiet 1000 is dan geven we 1 door

Test benaming moet toch eerder zijn als kredietlimiet 1000 is dan geven we 1 door

  • More
  • CR-48
  • started review
[UST2167]VHIP: overname van kredietlimiet uit Admin naar AX
[UST2167]VHIP: overname van kredietlimiet uit Admin naar AX