Skip to content

Conversation

@Shaggy84675
Copy link

Fixes #176

  • Tested Changes using phpunit
  • Have read and followed the Contribution Guidelines

This is my first contribution on Github so hopefully I'm doing it right. I've implemented basic forgot password functionality. It was discussed on Discord and also I found this suggestion in issues as well. There's a new link "Forgot password?" on login page. Existing user needs to put their e-mail address into reset password form and to that e-mail there's sent an e-mail with link for password reset. That means there must be set up SMTP server (or any other supported driver for sending e-mails). If someone wants to disable this functionality, they can just comment/remove "forgot-password" and "reset-password" routes. It will also remove "Forgot password" link from login page.

Because there's now more pages using login blade, maybe it'd be a good idea to create master blade for "guests" as well.

@Pyker
Copy link
Member

Pyker commented Aug 17, 2022

It intentionally doesn't have password reset functionality.

@Mondanzo
Copy link

Mind explaining why there is intentionally no reset function?

@rtm516
Copy link
Contributor

rtm516 commented Oct 27, 2023

I can understand why you might not want it on certain private instances but no reason not to have it as a feature admins can turn on and off from env or settings

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.

suggestion: technicsolder login > retrieve pass

4 participants