OLMIS-6953: Removed physical-inventory-draft-watcher, passed a flag to the...

Activity

FEOLMIS-4174 6

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 30m 3 You're right, I corrected it
    Reviewer - Complete 16m    
    Reviewer - Complete 11m 2 Is the draft object available here? Shouldn't we pass the...
    Reviewer - Complete 16m 1 I think we also need to cache VVM status while offline.
    Total   1h 14m 6  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    Paulina Buzderewicz

    I think we also need to cache VVM status while offline.

    I think we also need to cache VVM status while offline.

    Aleksandra Sołtys

    You're right, I corrected it

    You're right, I corrected it

    /src/stock-physical-inventory-draft/physical-inventory-draft-watcher.factory.js Deleted
    Open in IDE #permalink
    /src/stock-physical-inventory-draft/physical-inventory-draft-watcher.factory.spec.js Deleted
    Open in IDE #permalink
    /src/stock-physical-inventory-draft/physical-inventory-draft.controller.js Changed 4
    /src/stock-physical-inventory-draft/physical-inventory-draft.controller.spec.js Changed
    Open in IDE #permalink
    /src/stock-physical-inventory-draft/physical-inventory-draft.html Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time