Skip to content

Delegate sign_in_after_reset_password to resource class#5429

Closed
mattr wants to merge 1 commit intoheartcombo:mainfrom
mattr:feature/password-reset-configuration
Closed

Delegate sign_in_after_reset_password to resource class#5429
mattr wants to merge 1 commit intoheartcombo:mainfrom
mattr:feature/password-reset-configuration

Conversation

@mattr
Copy link
Contributor

@mattr mattr commented Dec 10, 2021

Allows resource class scopes to overrides the global configuration for sign in after reset password behaviour.

Allows resource class scopes to overrides the global configuration for sign in after reset password behaviour.
@carlosantoniodasilva
Copy link
Member

Thanks!

carlosantoniodasilva added a commit that referenced this pull request Feb 18, 2026
The config exists at the model/resource class from the registerable
module, but it was not being honored, instead we were directly relying
on the main Devise config.

Now this can be configured and honored per-model/resource class, as
expected.

This is similar to #5429 and `sign_in_after_reset_password` fix.
carlosantoniodasilva added a commit that referenced this pull request Feb 18, 2026
The config exists at the model/resource class from the registerable
module, but it was not being honored, instead we were directly relying
on the main Devise config.

Now this can be configured and honored per-model/resource class, as
expected.

This is similar to #5429 and `sign_in_after_reset_password` fix.
carlosantoniodasilva added a commit that referenced this pull request Feb 18, 2026
The config exists at the model/resource class from the registerable
module, but it was not being honored, instead we were directly relying
on the main Devise config.

Now this can be configured and honored per-model/resource class, as
expected.

This is similar to #5429 and `sign_in_after_reset_password` fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants