OLMIS-3243: Set NOT_APPLICABLE as default option if ENERGY_SOURCE is SOLAR

Activity

FEOLMIS-2122 18

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 55m 8 It will be done as a scope of OLMIS-3333.
    Reviewer - Complete 41m 10 I really like this.
    Total   1h 36m 18  
    #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

    Nikodem Graczewski

    Klaudia Pałkowska We should also make the status non-editable without require...

    Klaudia Pałkowska We should also make the status non-editable without required right.

    Klaudia Pałkowska

    It will be done as a scope of OLMIS-3333.

    It will be done as a scope of OLMIS-3333.

    Nikodem Graczewski

    We should introduce a fluent builder for InventoryItem class.

    We should introduce a fluent builder for InventoryItem class.

    /src/cce-catalog-item/catalog-item-builder.spec.js Added
    Open in IDE #permalink
    /src/cce-catalog-item/catalog-item.js Added
    Open in IDE #permalink
    /src/cce-edit-inventory-item/cce-edit-inventory-item.routes.js Changed 2
    /src/cce-edit-inventory-item/cce-edit-inventory-item.routes.spec.js Changed 2
    /src/cce-edit-inventory-item/edit-inventory-item.controller.js Changed 4
    /src/cce-edit-inventory-item/edit-inventory-item.controller.spec.js Changed 1
    /src/cce-inventory-item/cce-inventory-item.js Added 6
    /src/cce-inventory-item/cce-inventory-item.module.js Changed
    /src/cce-inventory-item/cce-inventory-item.spec.js Added
    /src/cce-inventory-item/inventory-item-builder.spec.js Added
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against