- Issue created by @RichardGaunt
- 🇦🇺Australia RichardGaunt Melbourne
What blocks this from happening:
1. Passwordless has a hook for redirecting users after logging in - at the moment this hook is called in the controller. This can be moved to a hook_user_login function
2. Sets the user reset message for users which happens after the `hook_user_login` is called - how to remove this message when logging in using the core module is what blocks this. - Assigned to antiorario
- 🇺🇸United States antiorario
Although I wouldn’t call this a proper bug, this issue raises a very good point, which boils down to letting Drupal do the heavy lifting and allowing Passwordless to do as little as possible, which has been my goal all along.
The main reservation I had was that making this change would inevitably introduce an extra step between clicking the login link and actually getting logged in (the good old “This is a one-time login…” form). But then I realized that this extra step would most likely be useful to fix 🐛 One-time login link processed by email clients / services Needs work , so I went for it. I’ll add the change straight to 2.0.x-dev.
-
antiorario →
committed fddd46d0 on 2.0.x
Issue #3374265: Overriding the user.reset route is unnecessary This...
-
antiorario →
committed fddd46d0 on 2.0.x
- Status changed to Fixed
about 1 year ago 11:22pm 30 September 2023 - 🇺🇸United States antiorario
This effectively breaks #2641148: Allow custom redirection after login → , but as RichardGaunt mentioned, it should be enough to implement
hook_user_login()
. Automatically closed - issue fixed for 2 weeks with no activity.