Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5643] CPConfig TBX ondersteunen in cache
  1. … 8 more files in changeset.
[2985] MVX aanpassing TOB montage fase 2
[2985] MVX aanpassing TOB montage fase 2
[2985] MVX aanpassing TOB montage fase 2

- NM TOB set eenheden en meenemers toevoegen

  1. … 11 more files 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-3082] [rvPVR] Uitbreiden OPCKenmerkenConverter nieuwe velden

- InkortingLijst toevoegen aan kenmerken zodat die ter beschikking zijn bij de productieaansturing, niet aan canonische waarde.

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

- KoppelingType zal voortaan per default altijd Movento zijn => converters en utils aangepast

- Ook die van Unishop aangepast => mapping bij KoppelingType uitgebreid met => Movento = M => Is makkelijkste oplossing om ketting converters op orde te houden

  1. … 7 more files in changeset.
Ook een klein stukje in Econ! (zie svn)

Ook een klein stukje in Econ! (zie svn)

[ICT-1693] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:
[ICT-1693] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:
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).

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

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.

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?

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

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

[ICT-1693][rvWV] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:

- BevatGeleiderBevestigingSchroeven toegevoegd in nodige dto'kes en kenmerkensets, alsook utils-kenmerkensets

- Conversie van BevatGeleiderBevestigingSchroeven voorzien waar nodig

- Vorige voorlopige change weer verwijderd, waarin BevatGeleiderBevestigingSchroeven hardcoded voor Alpnach op true gezet werd => wordt nu gezet in Econ en proper doorgegeven via de IngegevenKenmerken

  1. … 14 more files in changeset.
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.

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:

- Uitbreiding nodige dto-kes en Utils-klassen en convertoren en bijhorende UnitTests met AantalDwarsverdelingen en DwarsverdelingKleur

- Uitsluiting DwarsverdelingKleur in KenmerkDomeinPostProcessor om VHintra-configurator niet kapot te maken

  1. … 15 more files in changeset.