Web client: No progress indicator when renewing items

Bug #1755876 reported by Kerry Green
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned

Bug Description

This seems related to #1522638, but since that fix was released I'm reporting this as a new bug. The web client appears to have stalled when renewing large numbers of items.

tags: added: circ-checkout
Changed in evergreen:
status: New → Confirmed
assignee: nobody → Jessica Woolford (jwoolford)
Revision history for this message
Jessica Woolford (jwoolford) wrote :
tags: added: pullrequest
Revision history for this message
Jessica Woolford (jwoolford) wrote (last edit ):

Removing the pullrequest tag. More testing indicates the progress bar never closes if renewing from the Other/Special Circulations tab.

Edit: It actually seems to be that the progress bar never successfully closes, unless overridden by another invocation of the progress bar, which does close.

tags: added: needswork
removed: pullrequest
Revision history for this message
Jessica Woolford (jwoolford) wrote :

New branch, now with correct bug number, and an incrementing progress bar that closes correctly (thanks to the New Developers Group for that last bit!)

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jwoolford/lp1755876-renewal-progress-bar

tags: added: pullrequest
removed: needswork
Changed in evergreen:
assignee: Jessica Woolford (jwoolford) → nobody
Revision history for this message
Jessica Woolford (jwoolford) wrote :

Pushed up a fix that will start incrementing at 1 to the same branch, so that the increments match the number of items being renewed.

Revision history for this message
Susan Morrison (smorrison425) wrote :

I have tested this code and consent to signing off on it with my name, Susan Morrison, and my email address, <email address hidden>

tags: added: signedoff
Changed in evergreen:
milestone: none → 3.9.1
Changed in evergreen:
milestone: 3.9.1 → 3.9.2
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks, Jessica and Susan! I squashed the two commits into one and merged to rel_3_9 and above.

Changed in evergreen:
importance: Undecided → Low
status: Confirmed → Fix Committed
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.