Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- Toevoegen rol DwarsverdelingOnder

- Uitbreiding TaorboxModel met rol DwarsverdelingOnder

- Uitbreiden HalffabClassNameFactory met halffab DwarsverdelingOnder

- Deployklasse voor Selectiekenmerken en halffab-klasse aan te maken

- Builder van DwarsverdelingOnder gebruikt nu eigen rol

  1. … 8 more files in changeset.
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- Uitbreiding TaorboxModel met kenmerk DwarsverdelingType en rol DwarsverdelingDubbelHouder

- Uitbreiden TAORKenmerken met DwarsverdelingType

- Uitbreiden HalffabClassNameFactory met halffab DwarsverdelingDubbelHouder

- Toevoegen rol DwarsverdelingDubbelHouder

- Deployklasse voor Selectiekenmerken die halffab-klasse aangemaakt heeft

  1. … 5 more files in changeset.
Uitleg volgens Peter: Detailblad is een PDF en daar kun je die vhINVERS gebruiken, vhInvers is het markeren + bold zetten eigenlijk. Dit lukt enkel bij PDF's. Die onderdelenpickinglijst is een txt ...

Uitleg volgens Peter: Detailblad is een PDF en daar kun je die vhINVERS gebruiken, vhInvers is het markeren + bold zetten eigenlijk. Dit lukt enkel bij PDF's. Die onderdelenpickinglijst is een txt en daar kan volgens mij enkel die Bold werken

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!

Voor die LadeDieptes heeft Blum geen 70 kg Draagkracht bevestigd, vandaar.

Voor die LadeDieptes heeft Blum geen 70 kg Draagkracht bevestigd, vandaar.

Soms wordt "<B>" gebruikt en soms "<vhINVERS>" : ik veronderstel dat Peter weet wat het verschil is tussen beide. Of anders gezegd, waarom nu Bold en vroeger vhINVERS ? Er is mogelijks een verschil...

Soms wordt "<B>" gebruikt en soms "<vhINVERS>" : ik veronderstel dat Peter weet wat het verschil is tussen beide. Of anders gezegd, waarom nu Bold en vroeger vhINVERS ?
Er is mogelijks een verschil tussen OnderdelenPickingLijst en PdDetailBlad.
Vraag jij eens aan Peter? Of experimenteer zelf eens met een PdDetailblad.

N.B. : Volgens mij zorgt vhINVERS voor tekst in Bold en met grijze achtergrond.

Mijn vraag hier enkel om het te begrijpen : voor LadeDiepte 700 en 750 : blijft dit toch met de 60kg-geleider? Of is dat tijdelijk tot de stock opgebruikt is? Eventueel een beetje commentaar bij de...

Mijn vraag hier enkel om het te begrijpen :
voor LadeDiepte 700 en 750 : blijft dit toch met de 60kg-geleider? Of is dat tijdelijk tot de stock opgebruikt is?
Eventueel een beetje commentaar bij deze elseIF zetten.

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-2473] [rvWV] TAX - wijziging draagkracht MOVENTO 60kg -> 70kg:
[ICT-2473] [rvWV] TAX - wijziging draagkracht MOVENTO 60kg -> 70kg:
[ICT-2473] [rvWV] TAX - wijziging draagkracht MOVENTO 60kg -> 70kg:

- Korttekst gebruikt PPS-kenmerk en dat is door PM gewijzigd naar 70 tijdens uitfasering 60 voor bepaalde LadeDieptes.

- Uitbreiding enum Draagkracht in TaorboxModel

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

- Nog wat uitbreidingen met en defaultering op Movento (ook al wordt deze code wellicht niet meer gebruikt)

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

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 MaatwerkModel met rollen Dwarsverdeling en DwarsverdelingHouder

- Uitbreiding TAORKenmerken met AantalDwarsverdelingen en DwarsverdelingKleur

- Toevoeging halffabjes voor HalffabDwarsverdeling en HalffabDwarsverdelingHouder

- Uitbreiding BaseEnums met DwarsverdelingKleur

- Deployklasjes voor Selectie- en pps-kenmerken

  1. … 10 more files in changeset.