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

Disabling "Allow frontend edit" hides registration input #22

Open
muhpixel opened this issue Mar 7, 2019 · 2 comments
Open

Disabling "Allow frontend edit" hides registration input #22

muhpixel opened this issue Mar 7, 2019 · 2 comments

Comments

@muhpixel
Copy link

muhpixel commented Mar 7, 2019

I was wondering why this would be. The comment for the "Allow frontend edit" clearly states it's for making it editable in the user's account page, and this is how the Magento 1 version of this plugin works. However, I installed it today on Magento 2 (this was my first 3rd party module installed on Magento 2), and I noticed the register.phtml even has a condition for this:

if ($_username->isEnabled() && $_username->isEditable()):

So, having it disabled completely blocks it from being able to be set at registration. I'm not sure if this was intended, and it would be easy for me to adjust it if this was the intention. But was it?

@sylvainraye
Copy link
Collaborator

sylvainraye commented Mar 7, 2019

Hi @muhpixel ,

Thank you for your feedback.

Well, I agree it could be disappointing. I think the best should be to habe a separate configuration option to allow to create a username at registration or not. Completely separated from editing from user account.

What do you think ?

@muhpixel
Copy link
Author

muhpixel commented Mar 7, 2019

@sylvainraye Definitely. I know many sites that use a username type of community, prefer that most users choose their own username when they register, and then they do not wish for them to be able to edit them on their own afterwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants