Project

General

Profile

Task #1193

Redesign displaying messages for password policies

Added by Petr Michal over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Password policy
Target version:
Start date:
07/25/2018
Due date:
% Done:

100%

Estimated time:

Description

I've set up password policy for production use. The password policy rules are so long that I need to scroll to see form for password change.
Some messages are absolutely pointless (Show password policy name to users).
Most others overlap or duplicate different message.

The best solution for me will be a possibility to show message from new property instead of this.
The accuracy of the message will be responsibility of administrator.

password_policy.png (52.3 KB) password_policy.png Petr Michal, 07/25/2018 03:09 PM
pwdPreValidation.png (64.4 KB) pwdPreValidation.png Patrik Stloukal, 08/20/2018 11:14 AM
pwdPreValidation2.png (37.7 KB) pwdPreValidation2.png Patrik Stloukal, 08/20/2018 11:34 AM
preValidation.png (30.9 KB) preValidation.png special chars Patrik Stloukal, 09/26/2018 08:09 AM

Related issues

Related to CzechIdM - Task #1227: Allow use characters bases in password validationNew08/28/2018

History

#1 Updated by Radek Tomiška over 1 year ago

  • Tracker changed from Defect to Task

#2 Updated by Radek Tomiška over 1 year ago

  • Subject changed from Fix display of password policies to Redesign displaing messages for password policies

#3 Updated by Petr Michal over 1 year ago

Final design:
  • unnecessary points will be deleted: policy name, info about form, etc
  • duplications will be merged - special characters, enhanced control infromations, etc
  • all points will be merged into one box
  • box will be displayed under some help button

#4 Updated by Patrik Stloukal over 1 year ago

  • Assignee changed from Ondřej Kopr to Patrik Stloukal

#5 Updated by Patrik Stloukal over 1 year ago

So i redesign prevalidation messages, i fixed result code of prevalidation answer, deleted two lines (name of pwd policies and min rules to fulfil), then i redesign other rules to popup panel into one alert (bubble).
I created new component PasswordPreValidation, for easier use of this new feature.
I added this new component to pwd change, create new user and public pwd change.
Only i am not sure if anyone will notice this link, because it is blue on blue and it is almost as normal text... maybe add underline or different color?

Please look at this implementation and if you could send me some ideas, i'd be glad.

commits: https://github.com/bcvsolutions/CzechIdMng/commit/02aab236ec5fc50aa7eb6fe6c188cba2752da14c
https://github.com/bcvsolutions/CzechIdMng/commit/000f03703b6efb0e4388d27f6dd7e47c1e597765
https://github.com/bcvsolutions/CzechIdMng/commit/ad2d3524cc089b0945e4a7570945bb86a432a044

#6 Updated by Radek Tomiška about 1 year ago

  • Related to Task #1227: Allow use characters bases in password validation added

#7 Updated by Radek Tomiška about 1 year ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Radek Tomiška to Patrik Stloukal

I did test, review and refactoring (layout), notes consulted personally, summary:
- added prevalidation component will be removed (only one validation component will be used)
- raw html used in component will be refactored to react (usage of raw html was in component before, but i noticed it now)
- forbidden characters base will be shown in prevalidation (because is validated)
- Special characters (the same for forbidden characters) will be shown as one item in rule list (~li).

#8 Updated by Patrik Stloukal about 1 year ago

  • File preValidation.png preValidation.png added
  • Status changed from In Progress to Needs feedback
  • Assignee changed from Patrik Stloukal to Radek Tomiška
  • % Done changed from 80 to 90

ok so I resolved issues from last feedback, I removed raw html even in validation message.
Special and forbidden characters have different design for one pwd policy and more password policies, because merge especially for special chars is not ideal, so in this more pwd policies scenario there is list of special chars baseses and name of it's pwd policy. (preValidation.png)

Please look on this modified solution, I think it is now nice feature :D

commit: https://github.com/bcvsolutions/CzechIdMng/commit/1fc5f243b564bb83774c90e970cc6808c2bf956d
https://github.com/bcvsolutions/CzechIdMng/commit/ad476548bd9ceb285cffa4ef500df2a374cdc6cd
https://github.com/bcvsolutions/CzechIdMng/commit/08aced3cb8670a9ab5b541362d5316e2a568477f

#9 Updated by Radek Tomiška about 1 year ago

  • Status changed from Needs feedback to Closed
  • Assignee changed from Radek Tomiška to Patrik Stloukal
  • Target version set to Morganite (9.2.0)
  • % Done changed from 90 to 100

I like it, thx! It is much more nicer.

I merged it into develop.

Also available in: Atom PDF

Go to top