This is a list of all comments for CR-9.

Review Summary: No summary


----------------------------------------
File: ECON/impl/PM/Maatwerk/EdiConverter.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 17:16 
http://subversion02:8060/cru/CR-9#c25

  In de naamgeving van de klasse ontbreekt het feit dat het over
  TbxKenmerken gaat en ECONimpl is een root package
  cfr AXimpl en WSimpl
  
  Dus eigenlijk zou ECONimpl.PM.Maatwerk.TbxKenmerkenEdiConverter
  een correctere naam zijn --> natuurlijk is het dan noodzakelijk
  om de UT's hun naam te wijzigen

  Reply by Tom Vermeulen on 06 January 2017, 07:49
  > Dacht dat over een paar dagen/weken de converter ook LBX en
  > TAX ging moeten verwerken, daarom geen TBXKenmerken in naam.
  > De tests hebben wel TBX in hun naam. Andere convertoren
  > behandelen nu ook alle types schuifkes.

----------------------------------------
File: vhUnitTest/ECON/impl/PM/Maatwerk/EdiConverter/ConverteerEdiNaarEcon/TestTBX.cls.xml
Revision Comment by Jo Claes on 05 January 2017, 17:08 
http://subversion02:8060/cru/CR-9#c24

  Is eigenlijk een beetje muggenziften maar het zou nog dat
  tikkeltje beter zijn mocht de code er ongeveer zo uitzien
  
  do
  ..AssertEdiNaarEcon(##class(vhTest.Utils.APPS.EDI.common.dto.LadeTBX).StandaardLade(),##class(vhTest.Utils.ECON.PM.Maatwerk.dto.TbxKenmerken).StandaardLade())


---

ID: CR-9 http://subversion02:8060/cru/CR-9

Title: [UST2040] Vh4 TBX B.E.: e-Con naar Edi mapping van ingegeven kenmerken TBX
Statement of Objectives:


State: Closed

Summary: 

Author: Tom Vermeulen
Moderator: Tom Vermeulen
Reviewers: (0 active, 1 completed*)
     Jo Claes   (*)