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.

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

Already on GitHub? Sign in to your account

User authenticator needs to select from Active users only #198

Open
Gwalior-84 opened this issue Jun 24, 2021 · 0 comments
Open

User authenticator needs to select from Active users only #198

Gwalior-84 opened this issue Jun 24, 2021 · 0 comments
Labels
Bug Something isn't working Client: PNG Docs: not needed Issue does not require any changes to user documentation. Priority: Normal

Comments

@Gwalior-84
Copy link
Collaborator

Expected Behavior

When a new user logs in, the authenticator should check against only Active mSupply users.

Current Behavior

If the user has more than one entry in mSupply, one being Active and the other(s) having been synced from sync satellites, then the authenticator can check against an inactive version of the mSupply user.

Possible Solution

Steps to Reproduce (bugs only, delete if not applicable)

  1. Take system which has a user who normally works on a satellite server. Check to make sure that the user exists on the central server as an inactive user with no email address.
  2. Create the same user, using the same spelling, on the central server, with valid (non sussol) email address.
  3. Try to log the user in to the dashboard.
  4. Observe the error (sometimes? every time?)
@Gwalior-84 Gwalior-84 added Bug Something isn't working Docs: not needed Issue does not require any changes to user documentation. Priority: Normal Client: PNG labels Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Client: PNG Docs: not needed Issue does not require any changes to user documentation. Priority: Normal
Projects
None yet
Development

No branches or pull requests

1 participant