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

Activity

CR-2416 1

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author & Moderator 15m    
    Reviewer - Complete 16m 1 Wanneer meer dan 90% van de klassen verwijderd worden, da...
    Total   31m 1  
    #permalink

    Objectives

    Context: Stukken software in Admin om kosten die VH heeft aan klanten expliciet te berekenen en bewaren, met als doel deze op de KPI "Klant Product Analyse" te tonen.Dat KPI-KPA-bestand is echter al enkele jaren niet meer in gebruik (vervanging in Qlik).

    Te bekomen resultaat is dat die code verwijderd wordt.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    Wim Vermeulen

    Wanneer meer dan 90% van de klassen verwijderd worden, dan valt er nog weinig...

    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.

    /APPS/TRANSP/GLS/Shipment/Verwerker/ Deleted
    Open in IDE #permalink
    /APPS/.../Verwerker/GLSTransportKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../Verwerker/IndividueelKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/GLSTransportKostItem.cls.xml Moved
    Open in IDE #permalink
    /APPS/.../data/IndividueelKostItem.cls.xml Moved
    Open in IDE #permalink
    /APPS/TRANSP/.../Shipment/Verwerker.cls.xml Changed
    Open in IDE #permalink
    /APPS/common/KlantKost/ Deleted
    Open in IDE #permalink
    /APPS/common/KlantKost/enu/ Deleted
    Open in IDE #permalink
    /APPS/common/.../enu/KostType.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/KlantKost/impl/ Deleted
    Open in IDE #permalink
    /APPS/common/KlantKost/impl/KlantKost/ Deleted
    Open in IDE #permalink
    /APPS/common/KlantKost/impl/KlantKost/dto/ Deleted
    Open in IDE #permalink
    /APPS/common/.../KlantKost/dto/Factor.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../KlantKost/FactorGateway.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../impl/KlantKostService/ Deleted
    Open in IDE #permalink
    /APPS/.../KlantKostService/TransportFactorenInitiator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/KlantKost/impl/data/ Deleted
    Open in IDE #permalink
    /APPS/common/.../data/AfhalingKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../data/BezoekKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/BulkTransportKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/DoorgerekendKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/EnkelvoudigeFactor.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../impl/data/Factor.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/GLSTransportKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/GenotenKortingContantItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/IndividueelKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/LosTransportKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../impl/data/MaandKost.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/MeervoudigeFactor.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/MeervoudigeKlantSpecifiekeFactor.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../data/MonsterKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/OrderIngaveLoonKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../data/PostKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../data/RitTransportKostItem.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../impl/AfhalingKostCalculator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../impl/BezoekKostCalculator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../impl/GenotenKortingContantCalculator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../impl/KlantKostService.cls.xml Deleted
    Open in IDE #permalink
    /APPS/common/.../impl/KostCalculator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../impl/MaandKostAggregator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../impl/MonsterKostCalculator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../impl/OrderIngaveLoonKostCalculator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../impl/TransportKostCalculator.cls.xml Deleted
    Open in IDE #permalink
    /APPS/.../KlantKost/KlantKostService.cls.xml Deleted
    Open in IDE #permalink
    /Stat/enu/ Added
    Open in IDE #permalink
    /Stat/enu/KlantKostType.cls.xml Moved
    Open in IDE #permalink
    /Stat/OmzetKlantProductMaand.cls.xml Changed
    Open in IDE #permalink
    /vhDeploy/D20220103xKlantKostDataOpkuis.cls.xml Added
    Open in IDE #permalink
    /vhUnitTest/.../Verwerker/DataMKlantAPI.cls.xml Moved
    Open in IDE #permalink
    /vhUnitTest/.../Verwerker/TestGLSMeerdereBonnenInEenTransport.cls.xml Changed
    Open in IDE #permalink
    /vhUnitTest/.../Verwerker/TestGLSService.cls.xml Changed
    Open in IDE #permalink
    /vhUnitTest/APPS/common/KlantKost/ Deleted
    Open in IDE #permalink
    /vhUnitTest/APPS/common/KlantKost/impl/ Deleted
    Open in IDE #permalink
    /vhUnitTest/.../impl/KlantKostService.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../impl/TransportKostCalculator.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/DataMKlantAPI.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/KostUtils.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestAfhalingKosten.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestBezoekKosten.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestCommercieleOverdracht.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestDoorgerekendeKosten.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestFactor.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestGenotenKortingContant.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestMonsterKosten.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestOrderIngaveLoonKost.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestPortKosten.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestTransportKosten.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../KlantKost/TestTransportKostenDI2en3.cls.xml Deleted
    Open in IDE #permalink
    /zTryout/JLM/KlantKostDataLooper.cls.xml Added
    Open in IDE #permalink
    /vhCARETAKER.mac.rou Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against