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

Email address not check at registration #24959

Open
Varindien opened this issue Jun 14, 2021 · 7 comments
Open

Email address not check at registration #24959

Varindien opened this issue Jun 14, 2021 · 7 comments
Labels
Advanced parameters Component: Which BO section is concerned BO Category: Back Office Email Label: Which BO under menu is concerned Feature Type: New Feature Needs Specs Status: issue needs to be specified

Comments

@Varindien
Copy link

Varindien commented Jun 14, 2021

Is your feature request related to a problem?

When registering email address is badly check. eg email : test@gmailcom (missing dot)
this will succeed as registration but every module of shipping will be crashed ( tested with Colissimo, Chronopost, DPD, mondial relay) because of that. This is real world for now :)

Describe the solution you'd like

Maybe it will be interested so to add an option : An option to enforced verification email and avoid every module of shipping

image

@matks
Copy link
Contributor

matks commented Jun 14, 2021

Hello @Varindien there is no error here.

According to https://en.wikipedia.org/wiki/Email_address the email address test@gmailcom is valid. It should not be rejected by PrestaShop.

Emails can come in a lot of different shapes 😄 PrestaShop must accept them all

@Varindien
Copy link
Author

Hello @matks ,
I understand but in 99% of case, users will have an email with the dot. and in 100% of case it will break shipping module.
Maybe it will be interested so to add an option : An option to enforced verification email and avoid every module of shipping crashing ( tested with Colissimo, Chronopost, DPD, mondial relay) because of that. This is real world for now :)

What do you think?
thanks,

@hibatallahAouadni
Copy link
Contributor

Hello @Varindien

As @matks mentioned above xxx@gmailcom is a valid mail.

Maybe it will be interested so to add an option : An option to enforced verification email

But, what you proposed is a good feature and if you would work on it (Pull Request), we will really appreciate it 🎉
I'll modify your issue description to be as a feature request.

Thanks!

@hibatallahAouadni hibatallahAouadni added Advanced parameters Component: Which BO section is concerned BO Category: Back Office Email Label: Which BO under menu is concerned Feature Type: New Feature Needs Specs Status: issue needs to be specified Waiting for PM Status: action required, waiting for product feedback labels Jun 15, 2021
@hibatallahAouadni
Copy link
Contributor

Hello @Varindien

Thank you for your suggestion. The Product Team will take it into consideration for future developments.
Please be aware that there is no guarantee that this feature will be developed anytime soon. If this is important to you, we strongly suggest you consider hiring a specialist to help you create it.

If you are willing to implement this feature into PrestaShop as a Pull Request, the teams will be happy to help you by ensuring that it is appropriate and coherent with the rest of the project. If that is the case, please let us know!

Thank you

@Varindien
Copy link
Author

Hello @hibatallahAouadni ,

i am aware of delay which could be apply for new feature.
thanks you

@hibatallahAouadni
Copy link
Contributor

@Varindien if you submit a PR, the feature will be prioritized 😉

@Gipielle
Copy link

This is a problem that should be fixed before new features..

I think that will be great if merchants can upvote solutions, is i can see in this an also lots of post programmers don't care lot of merchants real problems

@marionf marionf removed the Waiting for PM Status: action required, waiting for product feedback label Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Advanced parameters Component: Which BO section is concerned BO Category: Back Office Email Label: Which BO under menu is concerned Feature Type: New Feature Needs Specs Status: issue needs to be specified
Projects
None yet
Development

No branches or pull requests

5 participants