Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ik had deze in de klassenaam zelf misschien ook "converter" bijgezet. Nu is dit HalffabItemsboom, maar is op die manier niet echt duidelijk dat het om een converter gaat zonder naar de folder te ki...

Ik had deze in de klassenaam zelf misschien ook "converter" bijgezet. Nu is dit HalffabItemsboom, maar is op die manier niet echt duidelijk dat het om een converter gaat zonder naar de folder te kijken (zelfde voor LBXKenmerken)

Zijn deze lijnen in commentaar nog relevant?

Zijn deze lijnen in commentaar nog relevant?

Samengesteld ipv SamenGesteld

Samengesteld ipv SamenGesteld

'Dotnet' -> Encoway

'Dotnet' -> Encoway

Rol doet niets, mag weg

Rol doet niets, mag weg

ProductApi is nergens gedefinieerd als input parameter, dus deze inject is niet nodig

ProductApi is nergens gedefinieerd als input parameter, dus deze inject is niet nodig

Ik zou dit niet als constructor porperty meegeven, want nu moet je overal waar je de VHConfigHelper wilt gebruiken, eerst hem initialiseren in de code zelf. Op die manier kan je moeilijker de vhcon...

Ik zou dit niet als constructor porperty meegeven, want nu moet je overal waar je de VHConfigHelper wilt gebruiken, eerst hem initialiseren in de code zelf. Op die manier kan je moeilijker de vhconfighelper injecteren.

Er is wel de method 'ZetVhConfig', maar die moet je dan eerst aanroepen vooraleer je de andere methods kan gebruiken. Ergens lijkt mij dat wat zot, omdat je dan snel fouten kan maken

Ik zou deze in een config item plaatsen

Ik zou deze in een config item plaatsen

In enu steken en hier ook naar verwijzen?

In enu steken en hier ook naar verwijzen?

Mijn gevoel is om dit ook in een aparte klasse te plaatsen en onder test te steken (ook al zal deze klasse dan vrij klein zijn)

Mijn gevoel is om dit ook in een aparte klasse te plaatsen en onder test te steken (ook al zal deze klasse dan vrij klein zijn)

Talen kunnen eventueel in een aparte enu?

Talen kunnen eventueel in een aparte enu?

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

[ICT-3982] Productcreatie in cache voor encowayconfiguraties
[ICT-3982] Productcreatie in cache voor encowayconfiguraties
[ICT-3982] Productcreatie in cache voor encowayconfiguraties
  1. ./Common/impl/OmleidingNaarEncowayBepaler
  2. … 24 more files in changeset.
[ICT-3672] Uitbreiding LBX BOMBOL in .NET

- Test mag berekingwijze encoway negeren

[ICT-2278] [rvWV] PM: Maatwerk: TAX: End2End-testen aanmaken, ter validatie van de builder(s):

- Prijscorrecties na laatste aanpassingen.

- Test voor omleiding naar dotnet van Taor gefixt

  1. … 3 more files in changeset.
[ICT-2278] [rvWV] PM: Maatwerk: TAX: End2End-testen aanmaken, ter validatie van de builder(s):

- TAOR heeft al builder in dotnet. Voorlopig de omleiding zelf wel nog op false houden

  1. … 1 more file in changeset.
TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- DiepteVerstellingSchroef => BuilderMoetWerken => voortaan altijd false vanwege bovenstaande + losse schroefjes geregeld door andere builder (SetSchroeven).

- HFSetSchroeven => ConditieDiepteVerstellingSchroefBepaler => Logica vóór vorige commit teruggezet, maar in comment for future reference, want...

=> Voortaan NOOIT schroefjes los meegeleverd, want weten niet welk front-materiaal de klant gebruikt en daarvan hangt keuze van schroefjes voortaan af

  1. … 2 more files in changeset.
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Nog wat UnitTest die irrelevant geworden zijn (Schroef- en DrevelKoppeling) uitgeschakeld.

- DiepteVerstelling wordt voortaan altijd los meegeleverd => geen boringen meer omdat nieuwe MoventoKoppeling => frezen ipv boren.

- DiepteVerstellingSchroef => BuilderMoetWerken => voortaan altijd false vanwege bovenstaande + losse schroefjes geregeld door andere builder (SetSchroeven).

- HFSetSchroeven => ConditieDiepteVerstellingSchroefBepaler => Logica i.g.v. DrevelKoppeling gespecifieerd, om afwijking met MoventoKoppeling te hebben. Bij DrevelKoppeling verandert er niets (maar gaat er nooit meer bijzijn).

=> I.g.v. MoventoKoppeling => Altijd schroefjes los meegeleverd, want ook DiepteVerstelling wordt dan los meegeleverd.

  1. … 3 more files in changeset.
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Alles is nu MoventoKoppeling => Sortering is alfabetisch en incl KoppelingType, dus volgorde nu anders

- 2 UnitTesten uitgeschakeld ivm SchroefKoppeling => ProductieRoute wegens irrelevant geworden => Is altijd MoventoKoppeling voortaan. We willen de code onder test echter nog niet verwijderen vanwege de manier waarop de infasering aangepakt werd (defaulting MoventoKoppeling op nogal vieze manier => zorgt ervoor dat deze UT nooit meer kan werken).

  1. … 1 more file in changeset.