Activity Delphi/ProScan/ProScan.identcache

11:21 am

Joannes Lemmens

More
CR-2828
started review

[ICT-4993][ProScan] Melding tonen indien toelevering voor kartonnen pallet is

9:06 am

Tom Vermeulen

More
Re: CR-2798
commented on review

Heb me enkel beziggehouden met de .pas bestanden

3:58 pm

Wordt deze gebruikt? Ik zie niet direct een verwijzing in de huidige klasse naar deze Service (kan er naast kijken). Indien niet in gebruik: uit de uses (gebruikt zinloze resources). Ook de file in...

Wordt deze gebruikt? Ik zie niet direct een verwijzing in de huidige klasse naar deze Service (kan er naast kijken). Indien niet in gebruik: uit de uses (gebruikt zinloze resources). Ook de file in \ProScan\WS\ wegsmijten in dat geval.

Als ie wel gebruikt wordt: dat is een serieuze dependency die geïntroduceerd wordt. Zou het er dan ook uitwerken

1:11 pm

Joannes Lemmens

More
CR-2798
started review

[ICT-3732] Manco deel 1: proboxen aan een pallet/LPN linken

1:11 pm

Joannes Lemmens

More
Re: CR-2798
commented on review

Lessons learned: opsplitsen in kleinere kaartjes... Gooi er maar gerust een extra reviewer (JBA) bij indien nodig.

Lessons learned: opsplitsen in kleinere kaartjes...
Gooi er maar gerust een extra reviewer (JBA) bij indien nodig.

1:45 pm

Joannes Lemmens

More
CR-2615
Closed

1:00 pm

Tom Vermeulen

More
CR-2615
finished reviewing

4:13 pm

Tom Vermeulen

More
Re: CR-2615
commented on review

je zou de wijzigingen in caché nog moeten toevoegen aan de review

4:04 pm

Joannes Lemmens

More
CR-2615
started review

[ICT-3707] XS Verpakkingsmateriaal - Planscan/ Proscan afmeldapplicatie

8:06 am

Tom Vermeulen

More
CR-2341
Closed

8:05 am

Tom Vermeulen

More
CR-2214
Closed

8:05 am

Tom Vermeulen

More
CR-2211
Closed

8:04 am

Tom Vermeulen

More
CR-2180
Closed

8:03 am

Tom Vermeulen

More
CR-2144
Closed

8:02 am

Tom Vermeulen

More
CR-2110
Closed

1:13 pm

Jelle Baele

More
CR-2341
finished reviewing

1:13 pm

Jelle Baele

More
Re: CR-2341
commented on review

Ik zou naar analogie met de overige code in de method 'GeefAantalOpPallet()' een try catch plaatsen en in de finally een '.Free'

9:02 am

Jelle Baele

More
Re: CR-2341
commented on review

Zou het niet properder zijn moest hier alle responses met hun '.Free' staan (Geen idee of dat oké is, maar lijkt mij van wel)? Dit stuk code wordt toch sowieso opgeroepen en dan staan ze allemaal o...

Zou het niet properder zijn moest hier alle responses met hun '.Free' staan (Geen idee of dat oké is, maar lijkt mij van wel)? Dit stuk code wordt toch sowieso opgeroepen en dan staan ze allemaal op 1 centrale plaats?

8:24 am