Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[VRB478] Afzonderen DOM naar APPS: LBX move classes/packages
  1. … 15 more files in changeset.
  • More
  • CR-594
  • finished reviewing
"Voor Wand ... " Het is jammer dat je de foute schrijfwijze hebt voortgezet. Ik weet dat je voorstander bent van consistente spelling, maar in dit geval (van nieuwe implementatie) had je gerust kun...

"Voor Wand ... "
Het is jammer dat je de foute schrijfwijze hebt voortgezet.
Ik weet dat je voorstander bent van consistente spelling, maar in dit geval (van nieuwe implementatie) had je gerust kunnen omschakelen.
Zeker als je weet dat een Klasse-rename bijna ONmogelijk is als het alleen om een hoofdletter gaat.
Met andere woorden, we zullen nooit meer van die "VoorWant" vanaf geraken

Anyway, het staat nu zo, dus laat maar staan. Geen onnodige tijd meer aan verliezen!
Verder is de volledige review OK, denk

[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):
[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):
[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):

- voorbereiding: voorwandtype bepaling afzonderen om code testbaar te maken

  1. … 10 more files in changeset.
[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):

- undo te veel gecommit

  1. … 1 more file in changeset.
[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):

- voorbereiding: lbx afzonderen om code testbaar te maken

- tests op converter toegevoegd

    • -0
    • +25
    ./VoorWandTypeBepaler.cls.xml
  1. … 10 more files in changeset.