Bug 32890 - Add DataTables to curbside pickups
Summary: Add DataTables to curbside pickups
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on: 30650
Blocks:
  Show dependency treegraph
 
Reported: 2023-02-06 17:00 UTC by Caroline Cyr La Rose
Modified: 2023-10-06 14:37 UTC (History)
1 user (show)

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


Attachments
Bug 32890: Add DataTables to curbside pickups (11.57 KB, patch)
2023-07-20 17:49 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 32890: Add DataTables to curbside pickups (11.64 KB, patch)
2023-07-20 18:59 UTC, Caroline Cyr La Rose
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Caroline Cyr La Rose 2023-02-06 17:00:30 UTC
This comes from https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32883#c2 I'm creating a new bug report as it was outside the scope of the other one.

I suggested adding Datatables to the curbside pickup module, to be able to manage the ordering and the columns in table settings.

- "To be staged" tab
- "Staged & ready" tab
- "Patron is outside" tab
- "Delivered today" tab
Comment 1 Owen Leonard 2023-07-20 17:49:35 UTC
Created attachment 153740 [details] [review]
Bug 32890: Add DataTables to curbside pickups

This patch modifies the curbside pickups page in the staff interface so
that each table is a DataTable with configurable columns.

The patch also replaces the patron name output with patron-title.inc for
consistency and to display names 'surname, firstname' for correct
column sorting.

To test, apply the patch and restart services.

- Enable the CurbsidePickup system preference if necessary.
- Properly testing requires testing data. You can run this command to
  REPLACE your curbside pickup data with sample data:

  bash <(curl -s https://gitlab.com/-/snippets/2572579/raw/main/test_curbside_pickups.sh)

- Go to Circulation -> Curbside pickups.
  - Test DataTable functionality under each tab: sorting, paging,
    filtering, column visibility, and export.
- Go to Administration -> Table settings -> Circulation -> Curbside
  pickup
  - Confirm that changes made to the configuration of each of the
    4 curbside pickup tables is correctly applied on the curbside
    pickups page.
Comment 2 Caroline Cyr La Rose 2023-07-20 18:59:52 UTC
Created attachment 153743 [details] [review]
Bug 32890: Add DataTables to curbside pickups

This patch modifies the curbside pickups page in the staff interface so
that each table is a DataTable with configurable columns.

The patch also replaces the patron name output with patron-title.inc for
consistency and to display names 'surname, firstname' for correct
column sorting.

To test, apply the patch and restart services.

- Enable the CurbsidePickup system preference if necessary.
- Properly testing requires testing data. You can run this command to
  REPLACE your curbside pickup data with sample data:

  bash <(curl -s https://gitlab.com/-/snippets/2572579/raw/main/test_curbside_pickups.sh)

- Go to Circulation -> Curbside pickups.
  - Test DataTable functionality under each tab: sorting, paging,
    filtering, column visibility, and export.
- Go to Administration -> Table settings -> Circulation -> Curbside
  pickup
  - Confirm that changes made to the configuration of each of the
    4 curbside pickup tables is correctly applied on the curbside
    pickups page.

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Comment 3 Caroline Cyr La Rose 2023-07-20 19:00:17 UTC
Much better! Thanks Owen!
Comment 4 Jonathan Druart 2023-09-01 12:30:15 UTC
Are not we then loosing the default sort we implemented on bug 32883?
Comment 5 Caroline Cyr La Rose 2023-09-01 12:50:37 UTC
Could the default ordering be the default, but then the user can change it if needed?
Comment 6 Caroline Cyr La Rose 2023-09-01 13:27:50 UTC
To be staged tab = default order should be on Pickup date/time soonest to latest

Staged & ready tab = default order should be on Pickup date/time soonest to latest

Patron is outside tab = default order should be on Pickup date/time soonest to latest

Delivered today tab = default sort should be on Deliver date/time soonest to latest (Jonathan mentioned this one was ordered by *desc*)
Comment 7 Owen Leonard 2023-10-06 14:37:11 UTC
I'm not sure I'm seeing the issue with the default sort under each tab. Can someone take another look?