Aleksandra Sołtys

Yes, but pending-offline-events.service.decorator should use getUserEventsFromStorage because we don't need there all the information that the getOfflineEvents returns

Yes, but pending-offline-events.service.decorator should use getUserEventsFromStorage because we don't need there all the information that the getOfflineEvents returns

Do you think you can extract something from the functions getOfflineEvents or combineResponses to make them a little smaller?

Do you think you can extract something from the functions getOfflineEvents or combineResponses to make them a little smaller?

The description of this function and the getUserEventsFromStorage are the same. However, this function returns more information

The description of this function and the getUserEventsFromStorage are the same. However, this function returns more information

This function is only used by getOfflineEvents but should also be used by pending-offline-events.service.decorator

This function is only used by getOfflineEvents but should also be used by pending-offline-events.service.decorator

OLMIS-7209: Corrected documentation of of the pending offline event service decorator
OLMIS-7209: Corrected documentation of of the pending offline event service decorator
You're right, I didn't notice it.

You're right, I didn't notice it.

OLMIS-7197: Added pending offline indicator in header
OLMIS-7197: Added pending offline indicator in header
OLMIS-7164: Disabled success notification after send issue/receive/adjustment in offline mode
OLMIS-7164: Disabled success notification after send issue/receive/adjustment in offline mode
Does this issue only occur in this one case? Maybe we should add this check to other methods as well

Does this issue only occur in this one case? Maybe we should add this check to other methods as well

What if the local database doesn't have all the requested records?

What if the local database doesn't have all the requested records?

We probably should also add the docId parameter

We probably should also add the docId parameter

I think you can add here similar code as you added in stock-card-summary-repository-impl.js to be sure the parameters programId and facilityId are given in the correct order to the docId

I think you can add here similar code as you added in stock-card-summary-repository-impl.js to be sure the parameters programId and facilityId are given in the correct order to the docId

OLMIS-7034: Fixed search for getDurationTimeThreshold
OLMIS-7034: Fixed search for getDurationTimeThreshold
OLMIS-7032: Fixed saving names with comma to csv file
OLMIS-7032: Fixed saving names with comma to csv file
I don't know if this is a good idea because some steps take several milliseconds. In this case, the results in seconds would have to be rounded to decimal places. On the other hand, results in seco...

I don't know if this is a good idea because some steps take several milliseconds. In this case, the results in seconds would have to be rounded to decimal places. On the other hand, results in seconds are more user-friendly. I think we might consider displaying the results in seconds in the OLMIS-7033 ticket where we will decide how we want to present the performance metrics

OLMIS-7032: Features durations counted and saved in a csv files
OLMIS-7032: Features durations counted and saved in a csv files
You're right, I corrected it

You're right, I corrected it

We don't need it, I haven't noticed that checkUnaccountedStockAdjustments is called by quantityChanged. Thank you Klaudia!

We don't need it, I haven't noticed that checkUnaccountedStockAdjustments is called by quantityChanged. Thank you Klaudia!

Draft object is available here and the data is correctly saved to local storage

Draft object is available here and the data is correctly saved to local storage

OLMIS-6953: Removed physical-inventory-draft-watcher, passed a flag to the saving-indicator instead...
OLMIS-6953: Removed physical-inventory-draft-watcher, passed a flag to the saving-indicator instead...
OLMIS-6925: Added reload page after back to physical inventory list with changed network state
OLMIS-6925: Added reload page after back to physical inventory list with changed network state