Add better errors for disabled account recovery #15117

jolheiser · 2021-03-23T02:12:42Z

jolheiser commented

2021-03-23 02:12:42 +00:00

#2247 was brought to my attention again in another Discord, so I figured I would attempt to remedy the error and give some slightly more useful information to the user/admin.

techknowlogick reviewed 2021-03-23 04:23:13 +00:00

templates/user/auth/forgot_passwd.tmpl

						
				@ -25,1 +25,3 @@

										<p class="center">{{.i18n.Tr "auth.disable_forgot_password_mail"}}</p>

										<p class="center">

											{{if $.IsAdmin}}

												{{.i18n.Tr "auth.disable_forgot_password_mail_admin"}}

techknowlogick commented

2021-03-23 04:23:12 +00:00

will this ever be true? Wouldn't IsAdmin require the user to be logged in?

jolheiser reviewed 2021-03-23 04:50:28 +00:00

templates/user/auth/forgot_passwd.tmpl

						
				@ -25,1 +25,3 @@

										<p class="center">{{.i18n.Tr "auth.disable_forgot_password_mail"}}</p>

										<p class="center">

											{{if $.IsAdmin}}

												{{.i18n.Tr "auth.disable_forgot_password_mail_admin"}}

jolheiser commented

2021-03-23 04:50:28 +00:00

Yes, and there is a "forgot password" link in user settings that would take them to this page.
Not sure what scenario it covers, exactly, though. Maybe forgot password but logged in with oauth?

Yes, and there is a "forgot password" link in user settings that would take them to this page. Not sure what scenario it covers, exactly, though. Maybe forgot password but logged in with oauth?