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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/BewaarMaatwerkLijn/Test.cls.xml
Revision Comment by Kurt Callebaut on 16 January 2017, 14:26 
http://subversion02:8060/cru/CR-23#c135

  deze test is intussen gewijzigd.

Revision Comment by Jo Claes on 12 January 2017, 15:16 
http://subversion02:8060/cru/CR-23#c81

  set Kenmerken =
  ##class(Tools.Mock.ArgumentMatchers.DataTransferObjectMatcher).%New([StandaardKenmerken])

Revision Comment by Jo Claes on 12 January 2017, 15:30 
http://subversion02:8060/cru/CR-23#c83

  Niet nodig om de bestaat via ExterneId op te roepen de
  GeefViaID is voldoende .. deze moeten dan wel de foutmelding
  encapsularen en zeggen dat er geen winkelkar bestaat met id.

Revision Comment by Jo Claes on 12 January 2017, 15:31 
http://subversion02:8060/cru/CR-23#c84

  GeefWinkelkarIdViaExterneId mag eigenlijk weg en het zou gewoon
  een GeefViaID mogen zijn waarvan je een offerte zal terug
  krijgen ( omdat voorlopig de repository geen notie hebben van
  wat ze moeten teruggeven , ( Strings, DataObjecten of Objecten
  ) op termijn zal het wel lukken als we de snelheid moeten
  aanpakken , maar voorlopig zou ik daar geen rekening met houden

Revision Comment by Jo Claes on 12 January 2017, 15:18 
http://subversion02:8060/cru/CR-23#c82

  EdiConverter en LegacyConverter etc... samen nemen in klasse
  WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Convert(EconStream)
  As APPS.EC.Winkelkar.dto.MaatwerkLijn

Revision Comment by Jo Claes on 12 January 2017, 15:33 
http://subversion02:8060/cru/CR-23#c85

  #dim Lijn niet nodig

Revision Comment by Jo Claes on 12 January 2017, 15:34 
http://subversion02:8060/cru/CR-23#c86

  method niet nodig -> standaardlade kenmerken gebruiken

Revision Comment by Jo Claes on 12 January 2017, 15:35 
http://subversion02:8060/cru/CR-23#c87

  Request mag in-line gemaakt worden in de test zodat de test op
  zich duidelijk is

Revision Comment by Jo Claes on 12 January 2017, 15:35 
http://subversion02:8060/cru/CR-23#c88

  #dim lijn niet nodig

Revision Comment by Jo Claes on 12 January 2017, 15:36 
http://subversion02:8060/cru/CR-23#c90

  niet nodig

Revision Comment by Jo Claes on 12 January 2017, 15:36 
http://subversion02:8060/cru/CR-23#c89

  niet nodig

----------------------------------------
File: WS/Vhisie4/Winkelkar/BewaarMaatwerkLijnRequest.cls.xml
Revision Comment by Jo Claes on 12 January 2017, 15:51 
http://subversion02:8060/cru/CR-23#c92

  Afhankelijk van het antwoord van intracto maar ik vermoed dat
  deze er terug af mogen

Revision Comment by Jo Claes on 13 January 2017, 08:37 
http://subversion02:8060/cru/CR-23#c93

  Property EconConfiguratie As TECH.String (XMLNAME =
  "IngegevenKenmerken") [ Required ];


---

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

Title: [UST2064] Vh4_SVC: BewaarMaatwerkLijn
Statement of Objectives:


State: Closed

Summary: 

Author: Kurt Callebaut
Moderator: Kurt Callebaut
Reviewers: (1 active, 0 completed*)
     Jo Claes