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

Enregistre le mot de passe après une connexion OpenID #1044

Merged
merged 5 commits into from
Oct 21, 2024
Merged

Conversation

thom4parisot
Copy link
Member

@thom4parisot thom4parisot commented Oct 14, 2024

Permet :

  • de modifier un mot de passe s'il est inexistant (en cas de première connexion via OpenID/Human-Id)
  • de se connecter soit avec un mot de passe soit avec Human-Id lorsque les deux sont configurés
  • de séparer le composant <Credentials /> et <UserInfos /> (plus facile à tester)
  • de rendre le formulaire accessible (avec des labels notamment)

fixes #552

Copy link

netlify bot commented Oct 14, 2024

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit 5a31d37
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/67163380c8fe5800084f23a1

Copy link

netlify bot commented Oct 14, 2024

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 1eedfe0
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/6716328f04c87a00085f4390
😎 Deploy Preview https://deploy-preview-1044--stylo-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -71,8 +71,12 @@ const userSchema = new Schema({
}
}, { timestamps: true });

userSchema.methods.comparePassword = function (password) {
return bcrypt.compare(password, this.password)
userSchema.get
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je pense que cette ligne est à supprimer ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bien vu 🦅 👁️ supprimé (ainsi que les autres console.log() 😅).

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

Successfully merging this pull request may close these issues.

Impossible de modifier le mot de passe d'un compte Stylo (non HumanID)
2 participants