Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-146
  • finished reviewing
[DEF565R] EDI bestellingen: doorgeven besteldatum

- Geen gemengd gebruik meer van Aanvraag als parameter of als property.

- Nog een Offerte.GeefId weggewerkt

- Wie goesting heeft kans desnoods ook Offerte overal als property gebruiken

  1. … 2 more files in changeset.
Wel raar dat er terug de ID wordt opgeroepen -> lijkt mij beter om code te krijgen alla Offerte.ZetHoofding(..OfferteHoofding) ipv het oproepen van de ExterneVerkoopService.ZetOfferteHoofding.

Wel raar dat er terug de ID wordt opgeroepen -> lijkt mij beter om code te krijgen alla Offerte.ZetHoofding(..OfferteHoofding) ipv het oproepen van de ExterneVerkoopService.ZetOfferteHoofding.

Wlip'kes

Wlip'kes

[DEF565] EDI bestellingen: doorgeven besteldatum
[DEF565] EDI bestellingen: doorgeven besteldatum
  • More
  • CR-118
  • finished reviewing
?? Bedoeling van dit veld ? Is een herhaling van de KlantNaam niet ok ???

?? Bedoeling van dit veld ? Is een herhaling van de KlantNaam niet ok ???

Kunnen we het nummer ook niet halen vanuit die andere Utils ???

Kunnen we het nummer ook niet halen vanuit die andere Utils ???

Goed idee om die data vanuit een andere Utils te halen

Goed idee om die data vanuit een andere Utils te halen

Misschien moeten we het iets explicieter maken dat het gaat over "EenBelg" of misschien nog beter "DenBelg" --> natuurlijk zal dat wel het gevolg hebben dat we "DenBelg" als begrip overal moeten ge...

Misschien moeten we het iets explicieter maken dat het gaat over "EenBelg" of misschien nog beter "DenBelg" --> natuurlijk zal dat wel het gevolg hebben dat we "DenBelg" als begrip overal moeten gebruiken.
Dan lijkt het mij nog beter om dat "DeBelgischeOnderneming" te nemen

!!!!!!

!!!!!!

?????

?????

[UST2215R] Labonorm EDI: leverdatum invullen

- Overal Offerte gebruiken in de UnishopBesteller ipv mengeling tussen offerte en offerteid

  1. … 4 more files in changeset.
awel, dat ik het eerlijk gezegd zelf totaal niet zou weten. Ik had eerst wat extra wijzigingen in deze klasse (niets van doen met deze lijn, maar soit), maar die zijn allemaal ongedaan gemaakt (beh...

awel, dat ik het eerlijk gezegd zelf totaal niet zou weten. Ik had eerst wat extra wijzigingen in deze klasse (niets van doen met deze lijn, maar soit), maar die zijn allemaal ongedaan gemaakt (behalve die on-the-flight natuurlijk). Zonder die typfout had ik trouwens gewoon een revert gedaan en was er van bovenstaande niets meer te zien. Het was een mysterie, het is een mysterie en zal ten eeuwige dagen een mysterie blijven denk ik.

Kwestie van geen $$$ElkeWaarde in mijn VerwachtMethodCall te moeten steken

Kwestie van geen $$$ElkeWaarde in mijn VerwachtMethodCall te moeten steken

Lijkt mij iets om de vhTest.Utils achter te laten en dat een beetje te algemenen

Lijkt mij iets om de vhTest.Utils achter te laten en dat een beetje te algemenen

Waarom wordt hier een DummyDto in de constructor meegegeven ?? en wat met de rest van de parameters ?

Waarom wordt hier een DummyDto in de constructor meegegeven ?? en wat met de rest van de parameters ?

Niet nodig om iedere keer hetzelfde opnieuw te testen..( DRY )

Niet nodig om iedere keer hetzelfde opnieuw te testen..( DRY )

Mock en Indien ??

Mock en Indien ??

Verwacht met Elke waarde ?

Verwacht met Elke waarde ?

Ja

Ja

mag het formaat zomaar gewijzigd worden ??

mag het formaat zomaar gewijzigd worden ??

waarom deze wijziging ??

waarom deze wijziging ??

lap daar gaat onze piloot-klant

lap daar gaat onze piloot-klant

Waar is de bewaar naar toe ???? ( ofwel moet ik er ergens naast kijken )

Waar is de bewaar naar toe ???? ( ofwel moet ik er ergens naast kijken )

Hiervoor zou ik toch de FormatterAPI gebruiken kwestie dat het toch wel allemaal iets duidelijker leest

Hiervoor zou ik toch de FormatterAPI gebruiken kwestie dat het toch wel allemaal iets duidelijker leest

[UST2215] Labonorm EDI: leverdatum invullen
[UST2215] Labonorm EDI: leverdatum invullen
[UST2215] Labonorm EDI: leverdatum invullen

- Enkel nog Offerte in gebruik

- en passant nog wat casing

  1. … 2 more files in changeset.