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

Non admin users logout #58

Closed
1 task
emteknetnz opened this issue Apr 13, 2021 · 4 comments
Closed
1 task

Non admin users logout #58

emteknetnz opened this issue Apr 13, 2021 · 4 comments

Comments

@emteknetnz
Copy link
Member

emteknetnz commented Apr 13, 2021

Split off from #46

Support for non-admin users to logout, which was the original reason for referencing logout_across_devices in the session-manager module

PRs

@brynwhyman
Copy link

@emteknetnz by 'non admin' users are you referring to users who have no access to the CMS (and their member UI)?

@emteknetnz
Copy link
Member Author

emteknetnz commented Apr 13, 2021

@brynwhyman that's correct

Typical way these types of members exist are for sites that allow website users to create an account via the frontend which will create a corresponding record in the Member table. User can log into their account, but that's also on the website frontend, not the CMS. This means the Member table will be shared by both CMS and website users.

I'm not sure what a scenario is where you'd have CMS users without access to the CMS

We could make a decent argument that supporting website account users via session-manager is out of scope.

@maxime-rainville
Copy link

My general view is that we shouldn't officially support this use case, while not precluding devs from implementing it on their website. We don't provide any front features for Member management out-the-box.

There's some third party modules that allow you to implement front-end member areas with profile pages. e.g.

I think it would make more sense for them to implement and maintain this kind of feature.

@emteknetnz
Copy link
Member Author

I'm closing this issue as it's unlikely to proceed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants