Comment Search

Bij clean code wordt in de "Select Case" VinylFiber ook expliciet vermeld.
Dan staat er bijvoorbeeld

 : is(MatMateriaal, 'CF'), '??' 

Wanneer de waarde toch iets anders zou zijn (bvb nieuwe waarde), dan zal het toch fout lopen in de volgende lijn, want de concat zal iets onbestaande opleveren.
Voordeel van "??" (of xx of "-" of zoiets) te gebruiken, is dat het meteen duidelijk is dat de fout in de eerste lijn zit, en niet in de concat.

ik wou eerst schrijven dat deze wijziging niks te maken heeft met deze story, maar je hebt dit heel terecht aangegeven in de commit message dat dit wel is. TOP!

Dus mijn conclusie is dan eigenlijk als volgt :
We mogen in het model niet zomaar characteristics verbergen ... we moeten steeds het 3D-model checken (ofwel jou op de hoogte brengen) !!!

Bij mijn eerste gedachten was de commentaar toch een beetje verwarrend.
Hier zou ik eerder spreken van "Uitsparing spoelbak - Bodem middelste gedeelte niet tonen in de 3D-vis"

De échte uitfrezing, (fysiek) op de onderkant van de bodemplaat uitgevoerd, staat immers hieronder, links+rechts, die comments wel oké.

comment : "Rugwand Staal : volledige lade, of linkse deel van Spoelbaklade"

Zijn er geen integratietesten van dat document?

Correct! Heb ik aangepast

Goed punt

ik zie de link niet tussen LadeKleurBuiten = "CSCL" en het material "MT_Blum_CSCL" (deze laatste vind ik nergens terug in de json of in het model)

Of ben je commits vergeten in de review hier?

'Dotnet' -> Encoway

Zijn deze lijnen in commentaar nog relevant?

Aangepast zoals besproken aan je bureau.

Ik was verward door het gebruik van de variabele "VerplaatsingZijkantRechts". Waarom gebruik je niet rechtstreeks "BodemBreedte"?
Dit is dezelfde waarde, maar in de context van "BevestigingBodemFront" is de link met Bodembreedte ook logischer, denk ik.

Mijn eerste idee was om $Number(Korting) te gebruiken, maar dit werkt niet omdat kortingen "?" en "" ook toegelaten moeten zijn.
Daarom heb ik heb ervoor gekozen om de logica lichtjes te veranderen, dit leek ook de meest cleane oplossing te zijn.

In plaats van waardes 0 en 1-100 toe te laten, laten we nu waardes 0-100 toe (dus ook een halve procent korting bijvoorbeeld). Dit omdat:

• Niemand ooit een fractie van een procent korting gaat geven
• Dit volgens Dries ook gewoon zo hoort te zijn

deze heeft er niet zo veel mee te maken zou ik zeggen. Probleem dat die mee in de commit zit?

Deze zou ik in de constructor injecteren en onder test zetten

Wlip mag weg

De groep specials lijkt me niet enkel voor spacetowers te zijn dus ik weet niet of deze conditie eigenlijk goed sluitend is voor de aangevraagde change?
Zie APPS.Halux.common.impl.ProductieGroepBepalerLBX

Converter in constructor initialiseren

$J zou ik vervangen door '$Job'?