CacheAdminA_trunk2010/vhDeploy/Maatwerk

TAOR

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 10
Last Commit: 25 Jun 15
Commits this week: 0
Total Lines of Code (LoC): 1.752
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 20 Sep 2022
10:42 am

Tommy Hebb

10:38 am

Tommy Hebb

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam v...

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam verschillend (cq. AantalDwarsverdelingen in APPS).

10:14 am

Tommy Hebb

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

Monday 19 Sep 2022
12:02 pm

Wim Vermeulen

12:02 pm

Wim Vermeulen

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logi...

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logica.
Maar ik geloof best, zonder het na te kijken, dat in deze klasse nog code staat die vertrekt van de ingegevenKenmerken i.p.v. halffabItem.

11:44 am

Wim Vermeulen

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties() hiervoor is geen "uitzondering nodig". Tijdens de uitbreidingsfase en review lijkt het me handig om te zien ...

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties()
hiervoor is geen "uitzondering nodig".
Tijdens de uitbreidingsfase en review lijkt het me handig om te zien dat beide properties zijn toegevoegd. Maar eenmaal in productie (en ondersteund door automatische testen) kan dit het standaard pad volgen.
Of zie ik iets over t hoofd?

11:10 am

Wim Vermeulen

Tekst correct? Zonder kar --> geen vakken, right?

Tekst correct?
Zonder kar --> geen vakken, right?

Tuesday 13 Sep 2022
3:25 pm

Tommy Hebb

Staat al in productie en is voordien al uitgebreid end-to-end getest.

Staat al in productie en is voordien al uitgebreid end-to-end getest.

3:24 pm

Tommy Hebb

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
Friday 02 Jul 2021
1:20 pm

Tom Vermeulen

9:06 am

Jelle Baele

Thursday 24 Jun 2021
4:43 pm

Jelle Baele

De review is een beetje een nest (reverts, code opnieuw toevoegen, ...). Bij onduidelijkheden, bel gerust. Heb hier ook al wat met Wim over gebeld en zei dat het in orde leek (heeft ook een beetje ...

De review is een beetje een nest (reverts, code opnieuw toevoegen, ...). Bij onduidelijkheden, bel gerust. Heb hier ook al wat met Wim over gebeld en zei dat het in orde leek (heeft ook een beetje bijgestuurd). De vhTestFiles zijn allemaal opgeschoven zodat de ProboxOverdoos .txt bestanden daar niet meer bijzitten.

De AansturingIncrementeel voor MVX lijkt in deze review niet te kloppen, maar dat is reeds opgelost in een ander kaartje naar aanleiding van het probleem van vrijdag 18/07. (zie https://vanhoecke.atlassian.net/browse/ICT-2408)

Ik ga dit zeker nog eens testen op accept (zodat niet hetzelfde voorvalt als bij het in productie plaatsen van het samengevoegd PIE) en ik heb al met Wim afgesproken om dit samen in productie te zetten als het zover is.

11:32 am

Jelle Baele

10:51 am

Jelle Baele

[ICT-2081] [rvTVE] Afdruk huidig proboxetiket uitschakelen
[ICT-2081] [rvTVE] Afdruk huidig proboxetiket uitschakelen
Tuesday 22 Jun 2021
3:26 pm

Wim Vermeulen

Monday 21 Jun 2021
4:41 pm

Peter Vercoutter

Wednesday 16 Jun 2021
6:12 pm

Wim Vermeulen

Aparte commits omdat ze niet allemaal samen konden gemerged worden.

Aparte commits omdat ze niet allemaal samen konden gemerged worden.

6:11 pm

Wim Vermeulen

[ICT-2399][rvPVR] Logging toevoegen aan de AansturingIncrementeel-klassen - method VoegToeLoggingNaExec() toegevoegd aan de basisklasse.
[ICT-2399][rvPVR] Logging toevoegen aan de AansturingIncrementeel-klassen - method VoegToeLoggingNaExec() toegevoegd aan de basisklasse.
Wednesday 07 Apr 2021
5:57 pm

Wim Vermeulen

Tuesday 06 Apr 2021
12:04 pm

Peter Vercoutter

Thursday 01 Apr 2021
5:17 pm

Wim Vermeulen

[ICT-2263][rvPVR] PAS: LBX aansturing incrementeel : deploy-klasse updaten (TAX)
[ICT-2263][rvPVR] PAS: LBX aansturing incrementeel : deploy-klasse updaten (TAX)
Tuesday 16 Feb 2021
10:00 am

Tommy Hebb

8:24 am

Tom Vermeulen

En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

Monday 15 Feb 2021
12:57 pm

Tom Vermeulen

12:50 pm

Tommy Hebb

Done!

Done!

12:23 pm

Tommy Hebb

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

12:20 pm

Tommy Hebb

Goed gezien :-s Bij deze aangepast!

Goed gezien :-s Bij deze aangepast!

Friday 12 Feb 2021
2:01 pm

Tom Vermeulen

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

2:00 pm

Tom Vermeulen

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

1:27 pm

Tom Vermeulen

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo