Skip to content
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

[make:registration-form] Add hash_property_path option to PasswordType #1555

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Seb33300
Copy link

@Seb33300 Seb33300 commented May 7, 2024

Fixes #1553

⚠️ I did not tested it

@seb-jean please help to test if it works as expected if you can

@Seb33300 Seb33300 force-pushed the registration-password-hash branch from 3977f91 to 7ee9197 Compare May 7, 2024 10:08
@seb-jean
Copy link
Contributor

seb-jean commented May 7, 2024

I just tried it and it works for me!

@seb-jean
Copy link
Contributor

seb-jean commented May 7, 2024

It's super interesting, it allows you to remove business logic from the controller.

@jrushlow jrushlow added Feature New Feature Status: Needs Review Needs to be reviewed labels May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New Feature Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[make:registration-form] Add hash_property_path option to PasswordType
3 participants