-
Notifications
You must be signed in to change notification settings - Fork 0
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
14 manage multiple sessions with user auth #16
14 manage multiple sessions with user auth #16
Conversation
Thanks ! |
Might be worth pulling upstream changes before merging (i.e. functional component). |
@axelboc and @antolinos, my apologies I managed to loose a PR from Axel two weeks ago. That was why I was a bit surprised yesterday @antolinos, (and was looking for things that I could not find). I reopened the PR on Again really sorry for the confusion, thanks for the PR's |
@marcus-oscarsson No problemo! |
ab84f66
to
d2f2414
Compare
9b21b20
to
e8d3ce5
Compare
Alright, so I thinks what @axelboc did already solves the problem, as you can see on:
So It only leaves out the commit the removes the user from the internal LIMS structures when we logout. If that makes sense. |
Thanks again, and my apologies for my clumsiness :/ |
Great! |
The following list should now been populated with the shared sessions thanks to the emit done at AbstractLIMS