Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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:

- Alles is nu MoventoKoppeling => Utils-kenmerken waren al aangepast, nu deze nog

  1. … 3 more files in changeset.
De klasse wordt maar éénmalig gegenereerd en ik vind het niet correct om foutieve code te laten staan, ook al is het een gegenereerde klasse en ook al wordt deze overridden in een andere klasse. Di...

De klasse wordt maar éénmalig gegenereerd en ik vind het niet correct om foutieve code te laten staan, ook al is het een gegenereerde klasse en ook al wordt deze overridden in een andere klasse. Dit kan enkel maar verwarrend werken en kans geven op fouten als anderen de gegenereerde gaan gebruiken.

SpecifiekeProductieWijze wordt nergens in de code gebruikt, dan zouden we de rest ook moeten refactoren naar specifieke...

SpecifiekeProductieWijze wordt nergens in de code gebruikt, dan zouden we de rest ook moeten refactoren naar specifieke...

Standaardbinnenlade wordt getest met ---> CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems()) maw gebruikt standaardlade en binnenlade bevat geen setschroeven.

Standaardbinnenlade wordt getest met ---> CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems()) maw gebruikt standaardlade
en binnenlade bevat geen setschroeven.

  • More
  • CR-91
  • finished reviewing
Waarom is er geen versie voor FlatpackV1 ??

Waarom is er geen versie voor FlatpackV1 ??

Waarom is deze niet toegevoegd bij de StandaardLadeItems()

Waarom is deze niet toegevoegd bij de StandaardLadeItems()

Volgens de overlevering mag je geen Generated klassen aanpassen , maar moet je daar een afgeleide klasse van maken en daar dan de specifieke implementatie in overschrijven

Volgens de overlevering mag je geen Generated klassen aanpassen , maar moet je daar een afgeleide klasse van maken en daar dan de specifieke implementatie in overschrijven

HeeftSpecifiekeProductieWijze lijkt mij toch een betere naamkeuze..

HeeftSpecifiekeProductieWijze lijkt mij toch een betere naamkeuze..

  • More
  • CR-91
  • started review
[UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set
[UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set
[UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set

- unitTesten aangepast. Set schroeven wordt nu toegevoegd aan gewonen lade. (integratietesten nog te corrigeren.-> prijswijzigingen)

  1. … 2 more files in changeset.
[UST1551] TAX VR: Rugwandhouders ifv Rug.concept ipv Lade.concept

- correctie: builder RugWandHouder moet wel werken bij verlaagde rug ofwel concept Baliko

  1. … 1 more file in changeset.
[UST1273] TAX duocolor: prijs: halffabbuilder en reworkfactor

- cleanup lege lijn

[UST1249] TAX LC: rug/bodem/zijkant/front: juiste items in boom

- Kantenband opgesplitst

- BouwsteenCreatie aanpassing

  1. … 41 more files in changeset.
[UST1249] TAX LC: rug/bodem/zijkant/front: juiste items in boom

- Rol rug is terug

  1. … 40 more files in changeset.
[UST1249] TAX LC: rug/bodem/zijkant/front: juiste items in boom

- ProjectSettings - IsTAORKoppeling actief - opkuis

  1. … 21 more files in changeset.
[UST1242] TAX-LC : HalffabItemsBoom is veranderd door IDynamischSamengesteldHF - Zijkant-subrollen voorlopig niet meer via RolAanwezig method.
[UST1242] TAX-LC : Zijkant SamengesteldHF maken: Zijkant opgesplitst in HF-Links/Rechts

- gebruik van TAX Rol enum

[UST1242] TAX-LC : Zijkant SamengesteldHF maken: Zijkant opgesplitst in HF-Links/Rechts

  1. … 37 more files in changeset.
[UST1242] TAX-LC : Zijkant SamengesteldHF maken: ZijkantBestukking en ZijkantBevestigingVeer verplaatst naar SamengesteldHF Zijkant
  1. … 30 more files in changeset.