Bootstrap OPAC: add support for curbside pickup appointments

[Galen Charlton]

Per bug 1879983, the Bootstrap OPAC should be updated to support the interfaces to allow patrons to manage curbside pickup appointments.

Evergreen master

[Jason Boyer (jboyer)]

Here's a branch to do this: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads / working/user/jboyer/lp1895737_bootstrap_curbside

[Jennifer Bruch (jbruchpails)]

Tried testing this today on Terran's Server.

Option to Schedule shows as expected once a hold was captured. Go to the Curbside page and it displays.

Got an Internal Server error message when clicking the Check Times button.

[Jennifer Bruch (jbruchpails)]

Patron info for testing today is :
username 99999336610
password normab1234

[Terran McCanna (tmccanna)]

Confirmed that it's not working on my Bug Squashing Week server, but it does appear to be working on Jason's pattypan server. I'm currently testing.

[Terran McCanna (tmccanna)]

The functionality works great, but there are a few some small styling / usability issues:

1 - Styling: The fields and buttons are scrunched (technical term) together and could use a little breathing room.

2 - Styling: The date box is unnecessarily wide (at least on my screen) with excess white space between the date and the calendar icon when active, and just a lot of excess white space in the box when inactive. Reducing the white space might help with #1.

3 - Usability: The date box allows you to pick a date in the past - if you submit it, it says that there are no times available and gives you a button to select another date. This isn't critical, but it's a little clunky and it would be nice if it didn't let you select a date in the past to begin with.

4 - Bug: If you do select a date in the past, you end up with two "Select another date" buttons instead of one (see screenshot).

5- Styling: The "Arrival Notes (vehicle description, etc)" column header is very long. Suggest breaking that into two lines or just saying "Arrival Notes" with a little "?" pop-up helper. This should also help with #1.

6 - Usability: After clicking "Alert staff of arrival", the button disappears, but it would be nice if there were some sort of confirmation message.

[Terran McCanna (tmccanna)]

Removing pullrequest until at least bullet 4 in comment #5 is resolved. As the others are styling issues, they could go into a new bug if needed.

[Jason Boyer (jboyer)]

I've given this another pass at https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jboyer/lp1895737_bootstrap_curbside2 / working/user/jboyer/lp1895737_bootstrap_curbside2

I've addressed the styling issues by not trying to wedge the tpac version into the boostrap opac as simplistically as I was, so this version also collapses better when used on a mobile device than before. Also I've taken a couple steps to avoid allowing you to choose a date in the past, and there's now a message after clicking the Alert Staff button.

[Terran McCanna (tmccanna)]

Looks and works great, Jason! My signoff is at

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1895737_bootstrap_curbside2_signoff

[Galen Charlton (gmc)]

Pushed to master, rel_3_7, and rel_3_6. Thanks, Jason and Terran!