Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wanneer meer dan 90% van de klassen verwijderd worden, dan valt er nog weinig te reviewen. De blijvende implementaties uiteraard, die zijn "Review O.K." Indien een klasse onterecht verwijderd zou z...

Wanneer meer dan 90% van de klassen verwijderd worden, dan valt er nog weinig te reviewen. De blijvende implementaties uiteraard, die zijn "Review O.K."
Indien een klasse onterecht verwijderd zou zijn, zal ik als reviewer hier helaas niet kunnen opmerken.
Er zijn wel nog enkele deploy-klassen die verwijzen naar de package "APPS.common.KlantKost". Deze kan je best ook nog opkuisen.

[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik.
[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik.
[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik:

- Fully qualified name veranderd (via 'MoveClass') om het beter te doen passen bij wat al bestond

    • -3
    • +3
    ./Shipment/Verwerker/TestGLSMeerdereBonnenInEenTransport.cls.xml
  1. … 6 more files in changeset.
[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik:

- Verhuizen van enkele KlantKost-data naar GLS waar ze mogelijks nog wordt gebruikt

    • -3
    • +3
    ./Shipment/Verwerker/TestGLSMeerdereBonnenInEenTransport.cls.xml
  1. … 4 more files in changeset.
[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik:

- Extra commit voor MoveClass: tests aangepast naar nieuwe locatie

    • -1
    • +1
    ./Shipment/Verwerker/TestGLSMeerdereBonnenInEenTransport.cls.xml
[ICT-1611] [rvWV] VKP: Klantkost: opkuis APPS.common.KlantKost package in caché, want niet meer in gebruik:

- do ##class(Tools.CacheSourceUtils).MoveClass("vhUnitTest.APPS.common.KlantKost.DataMKlantAPI","vhUnitTest.APPS.TRANSP.GLS.Shipment.Verwerker.DataMKlantAPI")

    • -0
    • +27
    ./Shipment/Verwerker/DataMKlantAPI.cls.xml
  1. … 2 more files in changeset.
[UW714] GLS: Nieuwe range voor Parcel instellen

- Na review: spaties toegevoegd.

    • -2
    • +2
    ./GLSv1/impl/UnitIdBepaler/Test.cls.xml
cosmetica: spaties rond = (ook in testje hieronder)

cosmetica: spaties rond = (ook in testje hieronder)

[UW714] GLS: Nieuwe range voor Parcel instellen
[UW714] GLS: Nieuwe range voor Parcel instellen
[UW714] GLS: Nieuwe range voor Parcel instellen

- Specifieke GLSEigenschappenMock toegevoegd, want reeds bestaande code+tests is niet eenvoudig aan te passen

- Testmethods voor automatisch overschakelen naar nieuwe Range

    • -0
    • +20
    ./GLSv1/impl/UnitIdBepaler/GLSEigenschappenMock.cls.xml
    • -0
    • +48
    ./GLSv1/impl/UnitIdBepaler/Test.cls.xml
[UW714] GLS: Nieuwe range voor Parcel instellen

- Opkuis niet-gebruikte mock-klasse (specifiek)

    • -0
    • +0
    ./CommunicatieService/CommunicatieServiceMock.cls.xml
[UW714] GLS: Nieuwe range voor Parcel instellen

- Mails bij "RangeBijnaOpgebruikt" : op 90% wordt nu ook een mail gestuurd --> bij 80% is meestal nog te vroeg om meteen actie te ondernemen.

    • -2
    • +3
    ./GLSv1/impl/UnitIdBepaler/Test.cls.xml
  1. … 1 more file in changeset.
[VRB] MAG: Transport: GLS-integratie: duidelijkere foutmelding naar gebruiker/ICT bij E001: a mandatory...
[VRB] MAG: Transport: GLS-integratie: duidelijkere foutmelding naar gebruiker/ICT bij E001: a mandatory...
[VRB] MAG: Transport: GLS-integratie: duidelijkere foutmelding naar gebruiker/ICT bij E001: a mandatory field is missing

- Aangezien velden ondersteund worden, maar niet meegegeven worden indien de property leeg is, geeft GLS een melding dat het/een veld ontbreekt. In de huidige werking kunnen we de E001 meldingen naar de gebruiker sturen met daarbij details voor de gekende velden die niet ingevuld zijn. Zoals postcode, land, telefoon, straat.

    • -1
    • +1
    ./CommunicatieService/TestResponse.cls.xml
  1. … 2 more files in changeset.
[UW479] Types uitklaren GLS - verschillende types door elkaar + 1 onbestaande klasse.
[UW479] Types uitklaren GLS - verschillende types door elkaar + 1 onbestaande klasse.
[UW479] Types uitklaren GLS - verschillende types door elkaar + 1 onbestaande klasse.

- Verwarring wegwerken. Onbestaande klassen die gebruikt worden bij GLS weggewerkt + juiste types bij method return types.

  1. … 6 more files in changeset.
[UST3741] GLS : afsluiten transport bij parcels .

Deze implementatie is bevestigd door GLS. Update sturen voor parcels is ok.

  1. … 1 more file in changeset.
[UST3741] GLS : afsluiten transport bij parcels .

In het onderzoek naar vroegtijdig in productie brengen van GLS gezien dat we afslutien van transport verder hadden moeten odnerzoeken. De berichten naar GLS blijken rekening te houden met het sluiten "DAYEND-END" van en transport, enkel voor Freight. Dus bij parcels is dat enkel "UPDATE" - we hebben nog bevestiging nodig van GLS over het hoe en wat bij parcels bij de nieuwe account voor de invulling van veld T090.. Deze commit geeft een mogelijke waarde aan, is gecommit om aan te geven waar de wijzigingen komen, maar is mogelijks niet definitief.

  1. … 1 more file in changeset.