Details
-
Story
-
Status: Closed (View Workflow)
-
P2
-
Resolution: Done
-
None
-
-
Core: F - Sprint 77, Core: F - Sprint 78
-
5
-
Prokopovych
Description
Purpose: We are going to have to revisit how we handle queue ordering when moving requests from one item to another now that we are going to allow manual reordering of request queues (per UXPROD-1242 and UIREQ-112, more specifically). Currently, when requests are moved from one item to another, they are placed in the destination queue FIFO by original request creation date. That is great in that it allows requests to be fulfilled in the order they are received. However, we now want to implement manual reordering of request queues and these features are not in conflict. The simplest way to resolve this conflict is to place moved requests at the end of the queue and ask the librarian to reorder them (they will often order them according to request creation date). See discussion notes below for discussion.
User story:
- As a circulation staff member who is moving a request from one item to another
- I want the moved request to appear at the end of the destination queue by default but an easy way to move it to another position in the queue (using manual queue reording (
UIREQ-112)) - So I can decide if I want to fulfill FIFO or make an exception for some reason
Scenarios
- Scenario
- Given I am moving a request from one item to another
- When the move has completed successfully
- Then:
- Success message should display (this is already working)
Works but so slooooow
- Move request modal should close (this is already working)
Works but so slooooow
- The request queue page with reorder capabilities (
UIREQ-112) should display for the destination queue and my moved request should appear at the end (last position) of the queueWorks but so slooooow
- Success message should display (this is already working)
- Scenario
- Given the request queue page with reorder capabilities
- When closed by clicking the X in the corner
- Then the moved request should display in view mode in the Requests app
Discussion notes:
How we came to decide on this change:
CB: Do we need to revisit the move requests functionality to make sure it still works? The current logic says that, when a request is moved from one item to another, it is slotted into the new queue according to request creation date so that the new queue is ordered FIFO. There are a couple of exceptions to this (i.e. a moved request can't supersede a request in position 1 when that request is a page or when it has already begun fulfillment). But generally speaking, that is the way the feature works. How can we reconcile these two features? Maybe we should say that, if a queue has been reordered, it gets a flag or some sort which tells the move request feature to just put the moved feature at the end of the queue. A popup could alert the user to this fact during the move. Thoughts?
Marc: When does a queue not become manually reordered?
- Could be when the queue empties, but that is very complex
- Could be a flag on a request that has been re-ordered (and that would apply too other requests whose position changed as the consequence of an explicit move). But then we would have to modify every request in the queue at the time of reorder (even if only the last two in the queue were swapped)
The simplest solution from a technical perspective is, that for every move, item would automatically go to the end of the queue and user would have to manually put it in the order they want
CB: Checked with RA SIG and Chalmers and both were okay with the simple approach described above.
TestRail: Results
Attachments
Issue Links
- clones
-
UIREQ-314 Reorder Request Queue on Dedicated Request Queue Page
-
- Closed
-
- defines
-
UXPROD-1242 Manually Reorder Request Queue (Dedicated Request Queue Page with Re-Order Capabilities)
-
- Closed
-
- has to be done after
-
UIREQ-112 Dedicated Request Queue Page with Reorder Capabilities
-
- Closed
-
- is blocked by
-
CIRC-455 UIREQ-334: Backend for revising logic for moving requests from one item to another
-
- Closed
-
-
CIRC-580 Error trying to move request when I have "Requests: All permissions" permission
-
- Closed
-
- relates to
-
CIRC-583 Move requests performance: Reuse fetched source and destination item and request queue during move request
-
- Open
-
-
CIRC-584 Move requests performance: Remove unnecessary reference records fetches when moving requests
-
- In Code Review
-
-
CIRCSTORE-176 Move requests performance: Improve performance of finding requests by itemId and status
-
- Open
-
-
UIREQ-389 Error reordering queue when I don't have Requests: Reorder queue option
-
- Closed
-
-
UIREQ-376 Unexpected "Request can not be moved above page request" error on move request
-
- Closed
-