vandelay: batch queue deletion no longer works

Bug #1067361 reported by Kathy Lussier
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned
2.2
Fix Released
Undecided
Unassigned
2.3
Fix Released
Undecided
Unassigned

Bug Description

Tested on Evergreen version 2.3 RC1 and on master

When using the new Vandelay batch queue deletion, I select queues to delete and then click the "Delete Selected" button. The screen refreshes, but the queues are still there.

Revision history for this message
Mark Cooper (markchristophercooper) wrote :

Quick fix is to make the delete calls synchronous so that the window.location.reload doesn't happen until the deletions are complete.

This worked for me:

To <email address hidden>:working/Evergreen.git
 * [new branch] lp1067361 -> user/mcooper/lp1067361

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

Mark, if you think your branch is ready for testing and acceptance, then please make sure you have signed off on your commits in git and add the pullrequest tag to this bug.

Thanks,

Changed in evergreen:
status: New → Confirmed
status: Confirmed → Incomplete
tags: added: pullrequest
Changed in evergreen:
status: Incomplete → Triaged
status: Triaged → Won't Fix
status: Won't Fix → Triaged
Revision history for this message
Kathy Lussier (klussier) wrote :

Works for me! Sign-off at:

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

While testing, I also noticed that the checkbox at the top of the queue list, which I assume is meant to select all queues on the page, isn't working. However, that's a different issue, and I'll submit a separate bug report on it.

Thanks Mark!

tags: added: signedoff
Revision history for this message
Ben Shum (bshum) wrote :

thanks Mark and Kathy!

Picked to master, rel_2_3 and rel_2_2.

Changed in evergreen:
milestone: none → 2.4.0-alpha
status: Triaged → Fix Committed
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.