Bug 35488 - Placing a hold on the OPAC takes the user to their account page, but does not activate the holds tab
Summary: Placing a hold on the OPAC takes the user to their account page, but does not...
Status: Pushed to oldstable
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Lucas Gass
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 35492 35495 35496
  Show dependency treegraph
 
Reported: 2023-12-05 15:28 UTC by Nick Clemens (kidclamp)
Modified: 2024-01-04 18:47 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.05.00,23.11.02,23.05.08


Attachments
Bug 35488: Open holds tab by default on opac-user.pl after placing a hold (2.56 KB, patch)
2023-12-05 16:41 UTC, Lucas Gass
Details | Diff | Splinter Review
Bug 35488: Open holds tab by default on opac-user.pl after placing a hold (2.60 KB, patch)
2023-12-05 17:19 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 35488: Open holds tab by default on opac-user.pl after placing a hold (2.66 KB, patch)
2023-12-12 19:38 UTC, Victor Grousset/tuxayo
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2023-12-05 15:28:39 UTC
To recreate:
1 - Place a hold on the opac
2 - Note after placing the hold the URL is:
    http://localhost:8080/cgi-bin/koha/opac-user.pl?#opac-user-holds
3 - Note the holds tab is not active
Comment 1 Lucas Gass 2023-12-05 16:41:58 UTC
Created attachment 159596 [details] [review]
Bug 35488: Open holds tab by default on opac-user.pl after placing a hold

1 - Place a hold on the opac
2 - Note after placing the hold the URL is:
    http://localhost:8080/cgi-bin/koha/opac-user.pl?#opac-user-holds
3 - Note the holds tab is not active
4 - APPLY PATCH and restart_all
5 - Try steps 1 - 3 again, this time the holds tab should be opened.
Comment 2 Owen Leonard 2023-12-05 17:19:26 UTC
Created attachment 159601 [details] [review]
Bug 35488: Open holds tab by default on opac-user.pl after placing a hold

1 - Place a hold on the opac
2 - Note after placing the hold the URL is:
    http://localhost:8080/cgi-bin/koha/opac-user.pl?#opac-user-holds
3 - Note the holds tab is not active
4 - APPLY PATCH and restart_all
5 - Try steps 1 - 3 again, this time the holds tab should be opened.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Victor Grousset/tuxayo 2023-12-12 19:38:59 UTC
Created attachment 159777 [details] [review]
Bug 35488: Open holds tab by default on opac-user.pl after placing a hold

1 - Place a hold on the opac
2 - Note after placing the hold the URL is:
    http://localhost:8080/cgi-bin/koha/opac-user.pl?#opac-user-holds
3 - Note the holds tab is not active
4 - APPLY PATCH and restart_all
5 - Try steps 1 - 3 again, this time the holds tab should be opened.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 4 Victor Grousset/tuxayo 2023-12-12 19:39:38 UTC
Works, makes sense, QA script happy, code looks good, passing QA :)
Comment 5 Katrin Fischer 2023-12-14 15:31:44 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 6 Fridolin Somers 2024-01-03 14:06:56 UTC
Pushed to 23.11.x for 23.11.02
Comment 7 Lucas Gass 2024-01-04 18:47:03 UTC
Backported to 23.05.x for upcoming 23.05.08