Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Naar deze enu's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

Naar deze enu's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

Naar deze dto's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

Naar deze dto's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

deze heeft er niet zo veel mee te maken zou ik zeggen. Probleem dat die mee in de commit zit?

deze heeft er niet zo veel mee te maken zou ik zeggen. Probleem dat die mee in de commit zit?

Als dit naar productie gaat, zeker ook de caretakerjob via de caché-systeembeheerportaal afzetten

Als dit naar productie gaat, zeker ook de caretakerjob via de caché-systeembeheerportaal afzetten

[ICT-3275] [rvTVE] Oud framework voor CubeIQ opkuisen
[ICT-3275] [rvTVE] Oud framework voor CubeIQ opkuisen
no_comment
    • -47
    • +0
    ./TRANSP/Offerte/impl/DHL/TariefRepository.cls.xml
    • -63
    • +0
    ./TRANSP/Offerte/impl/DHL/StapelingBepaler.cls.xml
  1. … 27 more files in changeset.
  • More
  • CR-288
  • finished reviewing
[ICT] - Routineload fix
[ICT] - Routineload fix
[ICT] - Routineload fix
zie hierboven

zie hierboven

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebru...

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.

aangezien deploy wordt er geen aanpassing doorgevoerd.

aangezien deploy wordt er geen aanpassing doorgevoerd.

Het ging hier over een bestaande parameter zonder type. ik heb het type toegevoegd.

Het ging hier over een bestaande parameter zonder type. ik heb het type toegevoegd.

  • More
  • CR-231
  • finished reviewing
  • More
  • CR-228
  • finished reviewing
Moet eigenlijk in /vhUnitTest/.../Service/GeefMogelijkeZendingen/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/GeefMogelijkeZendingen/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/BerekenPalletGegevens/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/BerekenPalletGegevens/Test.cls

hier en daar wat casing aanpassen

hier en daar wat casing aanpassen

$$$Not ipv ' Caché kennende is het ook veiliger om if (()&&()) te doen ipv if () && () (ook hieronder)

$$$Not ipv '
Caché kennende is het ook veiliger om if (()&&()) te doen ipv if () && () (ook hieronder)

Waarom die nieuwe parameter? Er wordt op het eerste zicht niets mee gedaan.

Waarom die nieuwe parameter? Er wordt op het eerste zicht niets mee gedaan.

  • More
  • CR-227
  • finished reviewing
Wat met klanten zonder transportkeuze?

Wat met klanten zonder transportkeuze?

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

ExactAantalKeer(1) om intentie duidelijk te maken

ExactAantalKeer(1) om intentie duidelijk te maken