This is a list of all comments for CR-2495.

Review Summary: No summary

General Comment by Tommy Hebb on 13 September 2022, 15:25 
http://subversion02:8060/cru/CR-2495#c4062

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


----------------------------------------
File: ECONimpl/PM/Maatwerk/KenmerkenConverter/AppsNaarEcon/Tax.cls.xml
Revision Comment by Wim Vermeulen on 19 September 2022, 11:41 
http://subversion02:8060/cru/CR-2495#c4069

  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?

  Reply by Tommy Hebb on 20 September 2022, 10:36
  > 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).

----------------------------------------
File: APPS/Halux/PPS/Activiteit/impl/TAOR/LijstVerwerkers/AssemblageKarOverzicht/DocumentInhoudBuilder/RijDataBepaler.cls.xml
Revision Comment by Wim Vermeulen on 19 September 2022, 11:03 
http://subversion02:8060/cru/CR-2495#c4067

  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.

----------------------------------------
File: APPS/Halux/PPS/Activiteit/impl/TAOR/LijstVerwerkers/AssemblageKarOverzicht/DocumentInhoudBuilderZonderKar.cls.xml
Revision Comment by Wim Vermeulen on 19 September 2022, 11:10 
http://subversion02:8060/cru/CR-2495#c4068

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

  Reply by Tommy Hebb on 20 September 2022, 10:14
  > Aangepast naar "Dwarsverdelingen te picken"


---

ID: CR-2495 http://subversion02:8060/cru/CR-2495

Title: [ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
Statement of Objectives:


State: Closed

Summary: 

Author: Tommy Hebb
Moderator: Tommy Hebb
Reviewers: (0 active, 1 completed*)
     Wim Vermeulen   (*)