Bug 42212 - Misleading column "Password expiration date" in module "Passwords (students)"
Misleading column "Password expiration date" in module "Passwords (students)"
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Wizards
UCS@school 4.1 R2
Other Linux
: P5 normal (vote)
: UCS@school 4.3 v4
Assigned To: Ole Schwiegert
Sönke Schwardt-Krummrich
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2016-09-01 14:40 CEST by Michel Smidt
Modified: 2018-07-04 18:09 CEST (History)
5 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.034
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments
Screenshot of module "Passwords (students)" (30.55 KB, image/png)
2016-09-01 14:40 CEST, Michel Smidt
Details
Screenshot of new interface (UCS@school 4.3v4) (15.79 KB, image/png)
2018-07-02 12:24 CEST, Sönke Schwardt-Krummrich
Details
Screenshot of new interface (UCS@school 4.3v4) (15.66 KB, image/png)
2018-07-02 12:54 CEST, Sönke Schwardt-Krummrich
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michel Smidt 2016-09-01 14:40:38 CEST
Created attachment 7960 [details]
Screenshot of module "Passwords (students)"

Reported by schooladmins through customer.

The enduser(s) was/were confused about the "Password expiration date" column in the UMC "Passwords (students)" module. If one reset passwords with the option "user has to change password on next login" the "Password expiration date" column shows the date from yesterday (see attached screenshots). 
The functionality works fine and this might be the right representation but from the user perspective he/she doesn't understand it.

It might be better to show a column with something like "recessed" yes/no
Comment 1 Florian Best univentionstaff 2017-06-28 14:56:53 CEST
There is a Customer ID set so I set the flag "School Customer affected".
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2018-04-27 11:18:38 CEST
We might need some discussion here. I see several solutions on this usability issue:

1) remove the column entirely

2) show only column "Password expired?" with values "yes"/"no"

3a) show column "Password expires in" with values "X days" resp. "expired"
3b) show column "Forced password change in" with values "X days" resp. "now"
Comment 3 Michel Smidt 2018-04-27 11:56:28 CEST
(In reply to Sönke Schwardt-Krummrich from comment #2)
> We might need some discussion here. I see several solutions on this
> usability issue:
> 
> 1) remove the column entirely
> 
> 2) show only column "Password expired?" with values "yes"/"no"
> 
> 3a) show column "Password expires in" with values "X days" resp. "expired"
> 3b) show column "Forced password change in" with values "X days" resp. "now"

Hi,
I prefer 3a)

Cheers,
Michel
Comment 4 Michel Smidt 2018-04-27 11:58:59 CEST
(In reply to Michel Smidt from comment #3)
> (In reply to Sönke Schwardt-Krummrich from comment #2)
> > We might need some discussion here. I see several solutions on this
> > usability issue:
> > 
> > 1) remove the column entirely
> > 
> > 2) show only column "Password expired?" with values "yes"/"no"
> > 
> > 3a) show column "Password expires in" with values "X days" resp. "expired"
> > 3b) show column "Forced password change in" with values "X days" resp. "now"
> 
> Hi,
> I prefer 3a)
> 
> Cheers,
> Michel

Ah no! I'am sorry. I prefer 3b!
Comment 5 Ole Schwiegert univentionstaff 2018-04-27 12:11:06 CEST
Personally I would prefer and also propose option 3a.
Option 1 and 2 remove or reduce information that we can/do already present which in my opinion are useful. 

A teacher could for example sort the list by that expiration date and check for students that have an already expired password or have to change it soon. That might come in handy when there is some project or work where the teacher wants to prevent the password-changing-and-resetting odyssey in advance or at least reduce the time necessary.

Regarding 3b I do not like the formulation "Forced password change". It might be my personal opinion but I also think "Password expiration date" or "Password expires in" are more mainstream terms most people should understand.
Comment 6 Ole Schwiegert univentionstaff 2018-05-09 12:41:14 CEST
A branch implementing 3b is provided under the name oschwieg/4.3/42212

Since this is my first time editing the backend it should be checked if my changes are compatible with univention best practices etc.
Comment 7 Ole Schwiegert univentionstaff 2018-05-25 12:30:47 CEST
Package: ucs-school-umc-users
Version: 14.0.2-4A~4.3.0.201805251229
Comment 8 Sönke Schwardt-Krummrich univentionstaff 2018-07-02 12:24:40 CEST
Created attachment 9581 [details]
Screenshot of new interface (UCS@school 4.3v4)
Comment 9 Sönke Schwardt-Krummrich univentionstaff 2018-07-02 12:54:16 CEST
Created attachment 9582 [details]
Screenshot of new interface (UCS@school 4.3v4)
Comment 10 Sönke Schwardt-Krummrich univentionstaff 2018-07-02 13:08:38 CEST
I did some small last-minute-changes to the wording. See attached screenshot.

ucs-school-umc-users (14.0.2-5)
ad0cbe325584 | Bug #42212: Small wording changes

ucs-school-umc-users.yaml
ff0463c062b8 | Bug #42212: update advisory
e3e928a3c5ab | Bug #42212: update advisory

OK: code change
OK: functional change
OK: tests
OK: changelog entry
OK: advisory
OK: package built and installable
Comment 11 Sönke Schwardt-Krummrich univentionstaff 2018-07-04 18:09:01 CEST
UCS@school 4.3 v4 has been released.

https://docs.software-univention.de/changelog-ucsschool-4.3v4-de.html

If this error occurs again, please clone this bug.