Activity CacheAdminA_trunk2010/WSimpl/Halux/AAP/AfdrukService

1:05 pm

Jelle Baele

More
CR-2887
started review

[ICT-5601] [rvJLM] Proboxinhoud etiket wordt soms op de verkeerde printer afgedrukt:

9:39 am

Tom Vermeulen

More
Re: CR-2740
commented on review

hier is in principe extra gedrag bijgekomen, dus zou er ook een testje dat moeten weergeven. Kan zijn dat ik niet in genoeg in detail naar de testen gekeken heb ook natuurlijk.

2:53 pm

Tom Vermeulen

More
Re: CR-2740
commented on review

If $$$HasLengfth(Werkost) (zie lijn 181)

2:52 pm

Tom Vermeulen

More
Re: CR-2740
commented on review

LadeRegistratieIterator ipv LadeRegistraties, dat maakt het lezen van de code duidelijker verderop. "LadeRegistraties" kan een list, een array, een iterator, ... zijn

2:51 pm

Tom Vermeulen

More
Re: CR-2740
commented on review

de opgeroepen functies gaan dus .Zoek* moeten heten (zie comment in repo)

2:50 pm

Een Geef geeft 1 product terug, een zoek mogelijks 0 tot meerdere. Implementatie is wel aangepast, maar de methodname is nog altijd een 'geef'. De oproeper hiervan verwacht dus 1 object terug te kr...

Een Geef geeft 1 product terug, een zoek mogelijks 0 tot meerdere. Implementatie is wel aangepast, maar de methodname is nog altijd een 'geef'. De oproeper hiervan verwacht dus 1 object terug te krijgen (maar dat is niet gegarandeerd door de gebruikte 'zoek'.
Naam (en bijhorende testen) aanpassen.
Idem met wijziging hieronder

4:23 pm

Jelle Baele

More
CR-2740
added Tom Vermeulen

4:21 pm

Jelle Baele

More
CR-2740
started review

[ICT-4601] [rvTVE] Afdrukken probox etiket robuuster maken

5:44 pm

Jelle Baele

More
CR-2704
Closed

12:32 pm

Peter Vercoutter

More
CR-2704
finished reviewing

12:32 pm

Peter Vercoutter

More
Re: CR-2704
commented on review

Hier zit nog een wlipke http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Hier zit nog een wlipke

10:31 am

Jelle Baele

More
Re: CR-2704
commented on review

Ik dacht eerst dat dit niet nodig was, maar heb het achter aan configsettings gestoken http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif.

Ik dacht eerst dat dit niet nodig was, maar heb het achter aan configsettings gestoken .

10:30 am

Jelle Baele

More
CR-2709
Closed

10:29 am

Jelle Baele

More
CR-2709
started review

[ICT-4302] Wijzigingen Caché en VHintra TST - Lijn 5:

9:50 am

Peter Vercoutter

More
Re: CR-2704
commented on review

Zou dit interessant zijn om achter een config setting te steken? Als dit de enige wijzigingen zijn voor L5 kan dit ook wel gewoon wachten met naar productie te gaan tot lijn 5 er is natuurlijk, up ...

Zou dit interessant zijn om achter een config setting te steken? Als dit de enige wijzigingen zijn voor L5 kan dit ook wel gewoon wachten met naar productie te gaan tot lijn 5 er is natuurlijk, up to you!

9:35 am