[UW352] Herberekenmaatwerk Visie 4

Activity

CR-136 3

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 17m 1 Ik zou dan voorstellen dat je zelf ook geen .ToArray gebr...
    Reviewer - Complete 21m 2 (2 defects) Volgens mij gaat deze test identiek hetzelfde testen dan ...
    Total   39m 3 (2 defects)  
    #permalink

    Objectives

    Jo, kan je dit al eens nazien en laten weten wat er nog moet gebeuren. (Los van de synchroniseer, die komt later aan de beurt.) -> zie cachebuilder. Ik zou voor synchroniseer lijncache ook niet public willen maken, maar visitor of dergelijke willen toepassen. suggesties ?

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /WSimpl/.../HerberekenMaatwerk/LijnCacheBuilder.cls.xml Changed
    /WSimpl/.../HerberekenMaatwerk/LijnCacheMatcher.cls.xml Deleted
    Open in IDE #permalink
    /WSimpl/.../HerberekenMaatwerk/LijnCacheValidator.cls.xml Deleted
    Open in IDE #permalink
    /WSimpl/.../Synchroniseer/Synchronisator.cls.xml Changed
    /WSimpl/.../WinkelkarService/HerberekenMaatwerk.cls.xml Changed
    /vhTest/.../HerberekenMaatwerk/LijnCacheBuilder.cls.xml Changed
    Open in IDE #permalink
    /vhTest/.../HerberekenMaatwerk/LijnCacheMatcher.cls.xml Deleted
    Open in IDE #permalink
    /vhTest/.../Synchroniseer/Synchronisator.cls.xml Added
    Open in IDE #permalink
    /vhTest/.../HerberekenMaatwerk/LijnCacheBuilder.cls.xml Changed
    /vhTest/.../HerberekenMaatwerk/LijnCacheMatcher.cls.xml Deleted
    Open in IDE #permalink
    /vhTest/.../Synchroniseer/Synchronisator.cls.xml Added
    Open in IDE #permalink
    /vhUnitTest/.../HerberekenMaatwerk/LijnCacheBuilder/
    Open in IDE #permalink
    /vhUnitTest/.../LijnCacheBuilder/Test.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../LijnCacheBuilder/TestBase.cls.xml Added
    Open in IDE #permalink
    /vhUnitTest/.../LijnCacheBuilder/TestGeefVerschillen.cls.xml Added
    Open in IDE #permalink
    /vhUnitTest/.../LijnCacheBuilder/TestValideerWinkelkarLijnen.cls.xml Added
    Open in IDE #permalink
    /vhUnitTest/.../HerberekenMaatwerk/LijnCacheMatcher/ Deleted
    Open in IDE #permalink
    /vhUnitTest/.../LijnCacheMatcher/Test.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../HerberekenMaatwerk/LijnCacheValidator/ Deleted
    Open in IDE #permalink
    /vhUnitTest/.../LijnCacheValidator/Test.cls.xml Deleted
    Open in IDE #permalink
    /vhUnitTest/.../HerberekenMaatwerk/Test.cls.xml Changed 3

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against