[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop-...

Activity

CR-126 7

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 16m 1 En als ik ze er expliciet bij zet, dan zijn ze waarschijn...
    Reviewer - Complete 12m 6 (3 defects) Ik zou echter alles testen door de APPS.EC.Winkelkar.impl...
    Total   28m 7 (3 defects)  
    #permalink

    Objectives

    Kan je eens nazien of dit functioneel correct is.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    Jo Claes

    Ik zou echter alles testen door de APPS.EC.Winkelkar.impl.ExterneWinkelkarSer...

    Ik zou echter alles testen door de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser ipv expliciet de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser.LBX aan te roepen.
    Nu ontbreken we de stap dat de KenmerkenAanpasser de juiste aanpassing doet indien het om LBX gaat.

    • Defect Ranking: Major Classification: Risk-prone
    /APPS/EC/.../KenmerkenAanpasser/LBX.cls.xml Added
    Open in IDE #permalink
    /APPS/.../ExterneWinkelkarService/KenmerkenAanpasser.cls.xml Changed
    Open in IDE #permalink
    /vhUnitTest/APPS/.../KenmerkenAanpasser/LBX/ Added
    Open in IDE #permalink
    /vhUnitTest/APPS/.../LBX/TestPasAan.cls.xml Added 6

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against