Evergreen

Add "email password reset" button to user editor

Bug #1264746 reported by Jeff Godin
40
This bug affects 9 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned
Evergreen 3.8-beta

Bug Description

When working with a patron over the phone who requires a password reset, an option in the staff client to generate a password reset request on the patron's behalf would be useful.

Currently, staff perform this task by copying the patron barcode to the clipboard, opening the catalog in a web browser, and submitting a password reset request as if they were the patron.

The user editor may be the best place for this new button, since this is where the existing "reset password" button lives.

The UI could provide a prompt to display the patron's current email address before triggering the reset request.

End result would be that the user would receive a password reset request e-mail with link to set a new password, just as if they had initiated the request from the catalog themselves.

It might also be useful to provide a clue to the A/T template that staff initiated the request, not the user themselves.

Jeff Godin (jgodin)
Changed in evergreen:
milestone: none → 2.6.0-beta1
Ben Shum (bshum)
Changed in evergreen:
status: New → Triaged
status: Triaged → In Progress
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.6.0-beta1 → 2.next
Revision history for this message
Terran McCanna (tmccanna) wrote :

+1

tags: added: patron webstaffclient
removed: staffclient usereditor
Revision history for this message
Jeff Godin (jgodin) wrote :

working branch for review at user/jeff/lp1264746_email_password_reset_button

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeff/lp1264746_email_password_reset_button

Revision history for this message
Jessica Woolford (jwoolford) wrote :

Jeff: If you get a chance to rebase your working branch, I would be interested in testing this.

Terran McCanna (tmccanna)
tags: added: needsrepatch pullrequest
Terran McCanna (tmccanna)
tags: removed: webstaffclient
Revision history for this message
Kyle Huckins (khuckins) wrote :

As it's been a while with no movement, in the interest of getting this on a test server I've done a quick rebase (looks like the only conflict was over a single piece of whitespace): https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/khuckins/lp1264746_email_password_reset_button_rebase

tags: removed: needsrepatch
Revision history for this message
Jennifer Bruch (jbruchpails) wrote :

I am attempting to test this.

I took patron 99999303411 and added my email, then clicked the new button to send reset link.

I get a toast message that it was submitted. However, no email so far.

Does the notice need to be turned on?

Revision history for this message
Kyle Huckins (khuckins) wrote :

The kcls-icelandic test server doesn't currently have outgoing email enabled, if you're testing it there. I'm currently looking into enabling it, if I can wrap my head around it. For what it's worth, the success popup only triggers on a successful response from the existing password reset API call, but it's probably good to be extra-certain

Terran McCanna (tmccanna)
Changed in evergreen:
assignee: Jeff Godin (jgodin) → nobody
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :

Works great! My sign off at:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1264746_email_password_reset_button_rebase_signoff

tags: added: signedoff
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master for inclusion in 3.8. Thanks, Jeff, Kyle, and Terran!

Changed in evergreen:
milestone: 3.next → 3.8-beta
status: In Progress → Fix Committed
Evergreen Bug Maintenance (bugmaster)
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.
Subscribing...

Other bug subscribers

Remote bug watches

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