Password Reset Button Text

Bug #1032646 reported by Tim Spindler
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

This is a small thing but we find that the text on the password reset button is deceptive. We have a lot of staff who enter a password for a patron and then click the reset button thinking what they type will be used to reset the password and they are not thinking it will apply a randome number or the phone number.

The text should probably be changed to something like:

"Generate Random Password" or "Generate Password"

Tags: pullrequest
Revision history for this message
Jason Stephenson (jstephenson) wrote :

There's no need to put wish list in the description. You can always set the importance to Wishlist when creating the bug, or rest assured that someone else will if they think it is a wish list item. (I might change my mind and make this "Low" importance later, not that I think a Wishlist item has less importance than Low necessarily.)

At any rate, I thought there had already been a branch that went in with different language for that button. I recall having discussion with Dan Scott on this very topic at some point in the past and testing code, etc.... (Am I hallucinating, again?)

Changed in evergreen:
importance: Undecided → Wishlist
summary: - Wish List: Password Reset Button Text
+ Password Reset Button Text
Revision history for this message
Ben Shum (bshum) wrote :

We made a local customization in our system and relabeled that button as "Random # Password". Consider my vote +1 to changing the text to something more effective.

For whatever it's worth, I'll push a copy of our changeset to working, though it doesn't take into account issues with i18n, etc. I think.

Working branch: user/bshum/random_password_wording

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/random_password_wording

Revision history for this message
Michael Peters (mrpeters) wrote :

Lets make it even more awesome and have 2 options:

* Reset Now
* Send Reset Email

Reset Now would do what it does now, and be used for users that don't have email addresses on file.

Send Reset Email would dispatch an email to the address on file with instructions on how to reset their password by clicking a link, thus allowing the patron to set their own password right from the get go, increasing the chances that they remember it!

Changed in evergreen:
status: New → Incomplete
Revision history for this message
Tim Spindler (tspindler-cwmars) wrote :

One of my issues with the "Reset Now" if it is still generating a random password, staff will still be confused thinking they enter a password and then click reset.

Instead of having the button set a random password. It should just reset the password if a new one is entered and alert staff if no password is set.

Workflow

1. Staff enters a new password for the patron
2. Staff click a button labeled "Reset Password"
3. Password button code checks that the text entered is indeed new and if not , gives a pop indicating that no new password is entered. If it is indeed a new password, then it generates an email to the patron if there is an email in the patron account if not it does nothing more than update the patron record.

Changed in evergreen:
status: Incomplete → Triaged
Revision history for this message
Kathy Lussier (klussier) wrote :

Thanks Ben! I showed this t a couple of people, who thought a "Generate Password" label might be a little clearer and also conforms with other sites that allow you to generate a random password. I've added a branch to working with the "Generate Password" label.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/generate_password

Kathy Lussier (klussier)
tags: added: pullrequest
Ben Shum (bshum)
Changed in evergreen:
assignee: nobody → Ben Shum (bshum)
milestone: none → 2.5.0-m2
Revision history for this message
Ben Shum (bshum) wrote :

Picked to master for a hopefully less confusing future!

Changed in evergreen:
status: Triaged → Fix Committed
Dan Wells (dbw2)
Changed in evergreen:
assignee: Ben Shum (bshum) → nobody
Ben Shum (bshum)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.