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

IBX-3407: Removed doubled validation from user account creation #36

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

ViniTou
Copy link
Contributor

@ViniTou ViniTou commented Aug 19, 2022

https://issues.ibexa.co/browse/IBX-3407

Password is validated as a part of Value validation inside \Ibexa\Core\FieldType\User\Type::validate

Note: this issues was occurring in every ezUser FT usage, so for example in create user in backoffice as well.

@ViniTou ViniTou force-pushed the ibx-3407-double-validation branch from 6f193b5 to 789046d Compare August 19, 2022 08:56
@ViniTou ViniTou requested a review from a team August 19, 2022 08:56
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@konradoboza konradoboza requested a review from a team August 19, 2022 08:58
@adamwojs adamwojs requested a review from a team August 19, 2022 09:03
@webhdx webhdx requested a review from a team August 19, 2022 09:06
@micszo micszo self-assigned this Aug 29, 2022
Copy link
Contributor

@micszo micszo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA Approved on Ibexa Commerce 4.2.1-dev.

@micszo micszo removed their assignment Aug 29, 2022
@ViniTou ViniTou merged commit 170c999 into main Aug 29, 2022
@ViniTou ViniTou deleted the ibx-3407-double-validation branch August 29, 2022 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants