Project

General

Profile

Actions
Copy link

Task #1193

closed

Redesign displaying messages for password policies

Added by Petr Michal over 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Patrik Stloukal
Category:
Password policy
Target version:
Morganite (9.2.0)
Start date:
07/25/2018
Due date:
% Done:

100%

Estimated time:
Owner:

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.

Files

Download all files
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 IdStory Identity Manager - Task #1227: Allow use characters bases in password validationNewOndřej Kopr08/28/2018

Actions
Actions
Copy link
 #1

Updated by Radek Tomiška over 6 years ago

  • Tracker changed from Defect to Task
Actions
Copy link
 #2

Updated by Radek Tomiška over 6 years ago

  • Subject changed from Fix display of password policies to Redesign displaing messages for password policies
Actions
Copy link
 #3

Updated by Petr Michal over 6 years 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
Actions
Copy link
 #4

Updated by Patrik Stloukal over 6 years ago

  • Assignee changed from Ondřej Kopr to Patrik Stloukal
Actions
Copy link
 #5

Updated by Patrik Stloukal over 6 years 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

Actions
Copy link
 #6

Updated by Radek Tomiška over 6 years ago

  • Related to Task #1227: Allow use characters bases in password validation added
Actions
Copy link
 #7

Updated by Radek Tomiška over 6 years 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).

Actions
Copy link
 #8

Updated by Patrik Stloukal about 6 years 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

Actions
Copy link
 #9

Updated by Radek Tomiška about 6 years 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.

Actions
Copy link

Also available in: Atom PDF