HFDiepteVerstellingSchroefv001.cls.xml

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:

- 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

    • -2
    • +4
    ./HFDiepteVerstellingSchroefv001.cls.xml
  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
    • +4
    ./HFDiepteVerstellingSchroefv001.cls.xml
  1. … 5 more files in changeset.
  • More
  • CR-132
  • finished reviewing
Het hergenereren van deze builder zal opnieuw deze conditie toevoegen. De conditie voor rol SetSchroeven in het TaorboxModel mag verwijderd worden

Het hergenereren van deze builder zal opnieuw deze conditie toevoegen. De conditie voor rol SetSchroeven in het TaorboxModel mag verwijderd worden

Kan in dit geval niet met de standaardlade gewerkt worden?

Kan in dit geval niet met de standaardlade gewerkt worden?

Heb je ook de TaorboxModel.Rollen.SetSchroeven aangepast? Dit kan mogelijks fout lopen bij hergenerer

Heb je ook de TaorboxModel.Rollen.SetSchroeven aangepast? Dit kan mogelijks fout lopen bij hergenerer

[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- Correctie conditie diepteverstelling is afhankelijk van gemonteerde lade en is een set L+R

    • -12
    • +4
    ./HFDiepteVerstellingSchroefv001.cls.xml
  1. … 1 more file in changeset.
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- Rol en halffab, builder voor DiepteVerstellingSchroef.

    • -0
    • +66
    ./HFDiepteVerstellingSchroefv001.cls.xml
  1. … 10 more files in changeset.