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

UI: remove password fields from register dialog #5591

Open
3 tasks
JayDi85 opened this issue Feb 14, 2019 · 2 comments
Open
3 tasks

UI: remove password fields from register dialog #5591

JayDi85 opened this issue Feb 14, 2019 · 2 comments
Labels
bug Bugs and errors GUI

Comments

@JayDi85
Copy link
Member

JayDi85 commented Feb 14, 2019

Current xmage code uses auto-generated passwords and sends it to emails on registration. No needs in password fields in register dialog. This confuses users.

TODO:

  • remove password fields;
  • checks registration with empty password;
  • fix missing "reset password" validate bug -- it's don't validate it by validatePassword proc;

shot_190214_064915

@JayDi85 JayDi85 added GUI bug Bugs and errors labels Feb 14, 2019
@kodemonster
Copy link

Are we removing both password fields.

@JayDi85
Copy link
Member Author

JayDi85 commented Feb 21, 2019

@kodemonster yes, register dialog don't need both password fields at all (password is auto-generated by server). But client must sent empty password to server as register method calls (for backward compatibility with old versions).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs and errors GUI
Projects
None yet
Development

No branches or pull requests

2 participants