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

Review Summary: No summary

General Comment by Tom Vermeulen on 22 October 2024, 11:04 
http://subversion02:8060/cru/CR-2821#c4434

  Hoe dan ook meer en kleinere commits.
  - klasse toegevoegd
  - filters aangepast
  - nieuwe klasse aangesproken
  - ...
  
  Kan alleen maar helpen voor als ge iets om zeep helpt tijdens
  ontwikkeling om te kunnen reverten zonder heel de nest te
  moeten terugdraaien. Helpt anderen ook om in de commitlog te
  zien wat er allemaal gebeurd is.Kan ook helpen om een taak die
  halverwege gestopt is (door andere prio's ofzo) eenvoudig door
  iemand anders te laten verderzetten.
  
  Soit, iets met gewoontes kweken enzo :)


----------------------------------------
File: UI/MB/UGLYPicking/dlgOrderListDeliverySelection.cls.xml
Revision Comment by Tom Vermeulen on 22 October 2024, 11:00 
http://subversion02:8060/cru/CR-2821#c4433

  Is het niet beter om hier een andere Locknaam te gebruiken? Ik
  heb niet teveel de code allemaal nagelezen, dus mogelijks zorgt
  het niet voor een probleem (of moet het expres dezelfde zijn).
  Hoe dan ook moet je de 'scope' van een lock zo beperkt mogelijk
  houden (maar het gedrag mag ook niet verkeerd kunnen gaan
  natuurlijk)


---

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

Title: [ICT-4568] Opsplitsing Orders obv verzendwijze/regio
Statement of Objectives:


State: Closed

Summary: 

Author: Peter Vercoutter
Moderator: Peter Vercoutter
Reviewers: (0 active, 1 completed*)
     Tom Vermeulen   (*)