Skip to content
This repository has been archived by the owner on Oct 6, 2022. It is now read-only.

Make the checkbox for Remember Me optional #189

Open
mela124 opened this issue Aug 24, 2018 · 3 comments
Open

Make the checkbox for Remember Me optional #189

mela124 opened this issue Aug 24, 2018 · 3 comments
Labels
enhancement New feature or request

Comments

@mela124
Copy link

mela124 commented Aug 24, 2018

Summary
This is mainly an option for FINN. The reason we have this checkbox is to ensure that the user is aware of such behavior and a way for them to opt out of it. FINN will provide these information another way (not during the flow) so they would want a way to hide the checkbox. See their proposal here: https://docs.google.com/document/d/1g3sHVTQDBPu1NXn17bKjgtTEQQVcgBRwD2btsLokd5g/edit#heading=h.ouuojhde7gj6

  • If it's hidden, then the user is considered still as opted-in as that's the default state of the checkbox
  • By default, the checkbox should be shown
@mela124 mela124 added the enhancement New feature or request label Aug 24, 2018
@3lvis
Copy link

3lvis commented Aug 29, 2018

I believe showing this checkbox on native apps is not a thing. Where did we get the idea to do this?

@mela124
Copy link
Author

mela124 commented Aug 29, 2018

Hi @3lvis, we discussed this with Thorild beforehand. It was needed to make sure the log in/sign up flow is GDPR-compliant. FINN suggested a different way to be GDPR-compliant but that is outside the flow that we provide thus we are thinking of making it optional. However, for those that won't do something similar as FINN, we still need to cover them therefore this functionality needs to be there.

We need to be able to be transparent to the user and provide them the opt out functionality that they will be stayed logged in on the app until they log out. In the future, if we do autologin across apps, this even becomes more critical.

@aliak00
Copy link
Contributor

aliak00 commented Sep 11, 2018

For this the config option should be added to IdentityUIConfiguration and persistence should be propagated from the PasswordCoordinator and the PasswordlessCoordinator

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants