-
-
Notifications
You must be signed in to change notification settings - Fork 834
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
"Send test email" button #1655
Comments
Are you suggesting to add this hack to the documentation? Is this a common support request? If so, can we fix the feature instead? (Sorry, asking out of cluelessness.) |
"Fix the feature" That would be possible by making mail configuration fool proof. I think it made sense to document this solution as it might happen more often than we like. |
"Fool proof" as in not accepting configuration that does not work? Not sure whether I would consider that necessary. (A "Send test mail" button might be easier...) That's like saying Flarum should display a pretty page when the database connection is misconfigured. 🤔 |
I only gave the solution, I never said that would be something we have to do 🤷♀️ Especially taking into consideration extensions modify the mail settings as well. As you proposed there are plenty of alternatives, even doing an automated test run before persisting the mail changes to the database might help. In any case this "hack" has already helped a few people. As such having it somewhere might be a wise idea, I agree the docs might not be the best solution, but I needed to put it somewhere in order not to forget about it. |
If @tobscure agrees, let's turn this into a "Send test email" button. :) |
Oh, and we could offer a console command for resetting admin's (or anyones?) password. |
Another good follow-up to #1169. |
This comment has been minimized.
This comment has been minimized.
The above 2 PRs should address both the test mail functionality and the console password reset functionality as requested in the discussion above. |
There is a way to reset your (admin) password without doing a full new reinstall.
Use the forgot password feature. In case the mail driver isn't fully working edit the
settings.mail_driver
value tolog
. Then run the password forgotten action again and checkstorage/logs/flarum.log
it will hold the full email contents.The text was updated successfully, but these errors were encountered: