Skip to content
This repository has been archived by the owner on Mar 10, 2024. It is now read-only.

[PROPOSAL] Deprecate passwordReset column in users table in favour of adding a new status to approved. #394

Closed
phillf opened this issue Apr 23, 2021 · 5 comments · Fixed by #376

Comments

@phillf
Copy link
Contributor

phillf commented Apr 23, 2021

Describe the solution you'd like

  1. Deprecate the passwordReset column in the users table.
  2. Rename the approved column to accountStatus.
  3. Introduce new status of 3 which would correspond to "User must reset password upon next login".
@phillf
Copy link
Contributor Author

phillf commented Apr 23, 2021

I am also thinking that that when a user is unsuspended their password should be changed to something completely random and a password reset upon next login should be forced.

This would require an email to be sent to the user upon suspension deprecation.

@ItsAGeekThing
Copy link
Member

I am also thinking that that when a user is unsuspended their password should be changed to something completely random and a password reset upon next login should be forced.

This would require an email to be sent to the user upon suspension deprecation.

This might be something that should be split into a separate issue/proposal.

@phillf
Copy link
Contributor Author

phillf commented May 13, 2021

The underlying problem with this is that the reset password facility isn't currently implemented.

@phillf
Copy link
Contributor Author

phillf commented Jun 15, 2021

I will create another issue for the implementation of a Password reset workflow.

Later on I will pull out the column in question as it doesn't serve a purpose.

@devotedsouls
Copy link
Contributor

This ticket has been split into two issues.

Ticket #398
Ticket #399

@opencad-community opencad-community locked and limited conversation to collaborators Jun 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.