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

Cannot add role for user if password has less than 8 characters #610

Closed
metas-mk opened this issue Sep 26, 2017 · 3 comments
Closed

Cannot add role for user if password has less than 8 characters #610

metas-mk opened this issue Sep 26, 2017 · 3 comments
Assignees

Comments

@metas-mk
Copy link
Member

Is this a bug or feature request?

Feature Request

What is the current behavior?

Belongs to metasfresh/metasfresh-webui-frontend-legacy#1218
The password Field is a mandatory Field. As soon as the user types at least 1 char, the mandatory logic is fulfilled and the "blue" borderline disappears. The record is not saved and the user does not receive an error.

Which are the steps to reproduce?

Check.

What is the expected or desired behavior?

Provide a solution.
Proposal: Could be a check how long the password entry is. If not 8 digits, and leaving field focus. Then show the field in red border and add error tooltip.

@teosarca WDYT?

@teosarca
Copy link
Member

@metas-mk IMHO we shall check, improve and/or fix following processes:

  • org.adempiere.user.process.AD_User_ChangePassword
  • org.adempiere.user.process.AD_User_ChangeMyPassword

Also, we shall hide the Password field.

@metasnw
Copy link
Member

metasnw commented Sep 26, 2017

there are also some suggestions here: metasfresh/metasfresh#2212

@teosarca
Copy link
Member

exactly that's the task we shall implement. Let's close this one.

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

No branches or pull requests

3 participants