User tests: Successful: Unsuccessful:
Pull Request for Issue #15786
Fixes the URL's for "Forgot your username" and "Forgot your password".
Also added target="_blank"
so the user doesn't have to manually navigate back to the admin page after.
Status | New | ⇒ | Pending |
Category | ⇒ | Modules Administration |
Labels |
Added:
?
|
sure, done
Thanks
After using this code in module, now the urls redirect to fronend forget password page.
Is it correct?
@RonakParmar One goes to &view=reset
and the other goes to &view=remind
I have tested this item
This isn't going to end up on a SEF friendly frontpage though. This is an issue we should solve as part of this PR
If you want to rewrite the router internals to be able to generate SEF URLs across applications, have at it. For now though, I don't see that as a blocker to this PR otherwise the links shouldn't have been added to the 3.x template.
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-08 09:34:01 |
Closed_By | ⇒ | wilsonge |
quite right about the norel stuff - wasnt thinking straight
but please remove the target blank. it makes it bad for a11y as you cant navigate back etc