stockmanagement

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
User should not see the home page, he should be redirected to the new view. If this is set to the login page user will not be able to open the application offline, because login does not work offline.

User should not see the home page, he should be redirected to the new view.
If this is set to the login page user will not be able to open the application offline, because login does not work offline.

shouldn't it be the home screen url or login page? Is it related to https://openlmis.atlassian.net/browse/OLMIS-7412 point 1?

shouldn't it be the home screen url or login page?

Is it related to https://openlmis.atlassian.net/browse/OLMIS-7412 point 1?

OLMIS-7315: Add manifest and service worker configuration for PWA
OLMIS-7315: Add manifest and service worker configuration for PWA
OLMIS-7315: Fix navigation on mobile

  1. … 5 more files in changeset.
done

done

yes, I think we should

yes, I think we should

you should remove this if it's not used

you should remove this if it's not used

done

done

done

done

Shouldn't we show the 'Home' tab in the menu?

Shouldn't we show the 'Home' tab in the menu?

if this is the default value it should be false

if this is the default value it should be false

I think you should create a method in the controller and move this logic there, also instead of checking the width you should use the window.matchMedia and check if it's standalone application

I think you should create a method in the controller and move this logic there,
also instead of checking the width you should use the window.matchMedia and check if it's standalone application

this should be false by default, so you don't need to add this option if it's false

this should be false by default, so you don't need to add this option if it's false

OLMIS-7318 Added the ability to unnecessary modules on mobile by using...
OLMIS-7318 Added the ability to unnecessary modules on mobile by using...
OLMIS-7318 Added the ability to unnecessary modules on mobile by using showInNavigationInLowResolutions

  1. … 7 more files in changeset.
This function is called by the service, and the service is called by factory in which we search for individual drafts in a loop by a given program and facility. Thanks to this, we do not have to ca...

This function is called by the service, and the service is called by factory in which we search for individual drafts in a loop by a given program and facility. Thanks to this, we do not have to call this physicalInventoryDraftCacheService in factory or routes. Only the physicalInventoryService knows and call physicalInventoryDraftCacheService

if there are no drafts in local storage, an empty list will be returned

if there are no drafts in local storage, an empty list will be returned

Then the name of this function could be "searchDrafts" or just "search".

Then the name of this function could be "searchDrafts" or just "search".

I guess we should search cached drafts for all programs for given facility. In this case searching just by facilityId would be enough if we want to display the list of available drafts.

I guess we should search cached drafts for all programs for given facility. In this case searching just by facilityId would be enough if we want to display the list of available drafts.

I think we should call this search when we open the list of drafts, not the particular draft.

I think we should call this search when we open the list of drafts, not the particular draft.

Could we refactor this file to use 'this.(...)' instead of local vars?

Could we refactor this file to use 'this.(...)' instead of local vars?

Please add 'this.' to avoid memory leaks.

Please add 'this.' to avoid memory leaks.

Could we use PhysicalInventoryLineItemDataBuilder here?

Could we use PhysicalInventoryLineItemDataBuilder here?