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

take out password field from user window and use process instead #2212

Closed
metasnw opened this issue Aug 21, 2017 · 3 comments
Closed

take out password field from user window and use process instead #2212

metasnw opened this issue Aug 21, 2017 · 3 comments

Comments

@metasnw
Copy link
Member

metasnw commented Aug 21, 2017

Is this a bug or feature request?

feature

What is the current behavior?

you can edit the password in user window
if you do not match password criteria you do not see the error message but the new button is not appearing

Which are the steps to reproduce?

follow http://docs.metasfresh.org/webui_collection/EN/NewUser.html
but use password with < 8 characters
NOK: no error message

try workaround:

  1. action "Reset password"
  2. NOK: does not work

What is the expected or desired behavior?

  1. password field is not shown at all
  2. process for reset password works in window user for all systemuser records
  3. update documentation
teosarca added a commit that referenced this issue Oct 3, 2017
metas-mk added a commit that referenced this issue Oct 3, 2017
Remove Mandatory Logic from password column in ad_user table

FRESH-3159 #2212
metas-mk added a commit that referenced this issue Oct 4, 2017
#2212 Take out password field from user window and use process instead
metas-mk added a commit that referenced this issue Oct 4, 2017
Adding new ReleaseNotes for Release 5.30
#2212 Take out password
field from user window and use process instead
@metasnw
Copy link
Member Author

metasnw commented Oct 4, 2017

tried on w101 to create a new user:
setting issystemuser and login does not allow to save user

error in console: "Das Passwort muss mindestens 8 Zeichen lang sein und darf keine Leerzeichen enthalten."

@metas-mk can you check please?

@metas-mk
Copy link
Member

metas-mk commented Oct 6, 2017

@metas-nw works for me.
image

@teosarca the adjustment in the process for empty "old" password doesn't seem to work
image

@metas-lc
Copy link
Contributor

  1. open user window
  2. create a new one: add firstname, lastname and login
    => it's saved. OK
  3. go to actions, "Change password", add new password + retype (make it 4 char length), start
    => Error: Das Passwort muss mindestens...
    Server error
    Das Passwort muss mindestens 8 Zeichen lang sein und darf keine Leerzeichen enthalten.
    OK
  4. repeat step 3 but make it 8 chars
    => password is set (checked the db)
  5. go to actions, "Change password", add new password + retype (don't set old password)
    => "Altes Passwort stimmt nicht überein"
  6. repeat 5 but set old password
    => password updated successfully OK
  7. open "Change password" process again
  8. write something random on each field, long press on the eye button
    => you can see what you wrote, when you let go it disappears OK

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

5 participants