physical-inventory-draft-toolbar.html

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
OLMIS-6900: Added message to save button about offline action

    • -1
    • +1
    ./physical-inventory-draft-toolbar.html
Our tester said that these buttons should return the message "This action can't be completed while offline."

Our tester said that these buttons should return the message "This action can't be completed while offline."

Hm, should it work this way? Because on requisition screen they are also disabled when offline.

Hm, should it work this way? Because on requisition screen they are also disabled when offline.

Thats because when they are turned off, they do not return messages that the operation cannot be performed offline

Thats because when they are turned off, they do not return messages that the operation cannot be performed offline

Why we don't want to have these buttons disabled while offline?

Why we don't want to have these buttons disabled while offline?

OLMIS-6900: Added message to buttons about offline action
OLMIS-6900: Added message to buttons about offline action
OLMIS-6900: Added message to buttons about offline action

    • -2
    • +2
    ./physical-inventory-draft-toolbar.html
typo?

typo?

Could we make this check more descriptive by extracting it to another boolean function?

Could we make this check more descriptive by extracting it to another boolean function?

OLMIS-6900: Fix for butons in offline mode and draft fetch from local storage

    • -3
    • +3
    ./physical-inventory-draft-toolbar.html
  1. … 3 more files in changeset.
For now there is a chance, that we will have two drafts with the same program and facility cached in our local storage, so the draft id can be needed here. But in this case we don't need searching ...

For now there is a chance, that we will have two drafts with the same program and facility cached in our local storage, so the draft id can be needed here. But in this case we don't need searching by program and facility, I guess.

The point is that when we have for example 2 drafts for the same facility and program in the list, when opening a draft we don't want to load random data, but we want to load data for the draft ind...

The point is that when we have for example 2 drafts for the same facility and program in the list, when opening a draft we don't want to load random data, but we want to load data for the draft indicated under a given id

Please update the changelog.

Please update the changelog.

It shouldn't be possible to have two drafts for the same program and facility.

It shouldn't be possible to have two drafts for the same program and facility.

Because when there are for example 2 drafts for the same program and facility and there is no id included, it can happen that incorrect data for another draft is loaded.

Because when there are for example 2 drafts for the same program and facility and there is no id included, it can happen that incorrect data for another draft is loaded.

Moving this was necessary?

Moving this was necessary?

Why we need draft id now?

Why we need draft id now?

This always returns me an empty list so I don't have reasons to select while offline.

This always returns me an empty list so I don't have reasons to select while offline.

I believe it was recently added... Why we don't need it anymore?

I believe it was recently added... Why we don't need it anymore?

Do we need a separate label for the offline draft?

Do we need a separate label for the offline draft?

This should be refactored to avoid code duplication.

This should be refactored to avoid code duplication.

OLMIS-6900 - Add possibility to open locally saved draft while offline
OLMIS-6900 - Add possibility to open locally saved draft while offline
OLMIS-6900: Add ability to open locally saved drafts offline

    • -3
    • +3
    ./physical-inventory-draft-toolbar.html
  1. … 9 more files in changeset.
https://review.openlmis.org/cru/#FEOLMIS-1923CFR-29468

https://review.openlmis.org/cru/#FEOLMIS-1923CFR-29468