Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Coldchain Equipment list shows equipment for all active stores on remote site #4235

Closed
lache-melvin opened this issue Jun 21, 2024 · 3 comments 路 Fixed by #4316
Closed

Coldchain Equipment list shows equipment for all active stores on remote site #4235

lache-melvin opened this issue Jun 21, 2024 · 3 comments 路 Fixed by #4316
Assignees
@lache-melvin
Labels
bug Something is borken Build Tested: Passed Issue has been build tested with all tests passed. Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release Team Piwakawaka Mark, James, Fergus Tester: Anusha
Milestone
V2.1.0

Comments

@lache-melvin
Copy link
Contributor

What went wrong? 馃槻

Have a remote site with 2 stores: Opua and Kawakawa

On central, create an asset for Kawakawa:

image

On remote, logged into Opua, can see Kawakawa asset?

Screenshot 2024-06-21 at 4 29 16鈥疨M

Expected behaviour 馃

I only see my store's assets when looking at my store...

If clients aren't really doing multi-store this is probably low priority though 馃槄

How to Reproduce 馃敤

Steps to reproduce the behaviour:

  1. Follow steps above 馃檲

Your environment 馃尡

  • Open mSupply Version: 2.1 RC 1
  • Legacy mSupply Central Server Version:
  • Platform:
  • Database type:
@lache-melvin lache-melvin added bug Something is borken needs triage needs daily triage New bug to be triaged by the release team labels Jun 21, 2024
@adamdewey
Copy link
Contributor

Good catch Lach茅!

@CarlosNZ CarlosNZ added Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. and removed needs daily triage New bug to be triaged by the release team labels Jun 23, 2024
@andreievg andreievg added Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release and removed needs triage Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. labels Jun 27, 2024
@andreievg andreievg added this to the v2.1.0-rc5 milestone Jun 27, 2024
@andreievg andreievg added the Team Piwakawaka Mark, James, Fergus label Jun 27, 2024
@lache-melvin lache-melvin self-assigned this Jun 27, 2024
@lache-melvin lache-melvin mentioned this issue Jun 30, 2024
Merged
9 tasks
@lache-melvin lache-melvin linked a pull request Jun 30, 2024 that will close this issue
4235 non-central stores should only see their assets #4316
Merged
9 tasks
@lache-melvin
Copy link
Contributor Author

Closed by #4316

@roxy-dao roxy-dao modified the milestones: v2.1.0-rc5, V2.1.0 Jul 2, 2024
@AnushaUp
Copy link
Collaborator

AnushaUp commented Jul 4, 2024 •
edited
Loading

馃И Testing

Tested in OMS V2.1.00-rc6

  • Have a remote site with at least two stores (both with vaccine module enabled)
  • On one store, add some cold chain assets
  • log in to the other store
  • You CANNOT see the assets from the first store
  • Log into central
  • You can still see the assets for all stores

@AnushaUp AnushaUp added the Build Tested: Passed Issue has been build tested with all tests passed. label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lache-melvin lache-melvin

Labels
bug Something is borken Build Tested: Passed Issue has been build tested with all tests passed. Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release Team Piwakawaka Mark, James, Fergus Tester: Anusha
Projects
None yet
Milestone
V2.1.0
Development

Successfully merging a pull request may close this issue.

4235 non-central stores should only see their assets msupply-foundation/open-msupply
7 participants
@CarlosNZ @adamdewey @andreievg @lache-melvin @nisha-dangol @roxy-dao @AnushaUp