Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Shopping cart delivery note #1357

Open
1 task
ccleeton opened this issue Dec 14, 2023 · 18 comments
Open
1 task

Shopping cart delivery note #1357

ccleeton opened this issue Dec 14, 2023 · 18 comments
Labels
dls-work-cycle Tickets designated for the next work cycle (limit 16) On Deck Next issues marked by product owners for working on. user experience

Comments

@ccleeton
Copy link
Collaborator

ccleeton commented Dec 14, 2023

Summary

We'd done some work to add a shopping cart delivery note, to warn people that the items will be paged (or are under restriction) when they arrive.

We found some problems with that work, and now we need to roll it back.

Acceptance Criteria

  • The commit d86c207 is rolled back, and no longer live in production.

First Step

git revert d86c207d6a206420c3dc84249a2c5ef4a766c0aa, then commit that and push it up.

Previous Ticket Info

Add the following text to items that would be viewed in the reading rooms.

These item(s) will be paged upon your arrival to the reading room.

This is to help researchers understand when they will be able to see items they have requested.
Related/similar to ticket #1154 #1244 & #1249

@hackartisan
Copy link
Member

👍🏻 one question, are we sure that "paged" is a clear word to patrons? Will they know what this means?

@ccleeton
Copy link
Collaborator Author

We do use that word fairly frequently. I see your point, but I do think it works for our purposes.
I also don't have a better word.

@faithc faithc added the On Deck Next issues marked by product owners for working on. label Jan 29, 2024
@ccleeton
Copy link
Collaborator Author

ccleeton commented Jan 29, 2024

other terminology options could be:
using active verbs
retrieve
deliver
bring
provide
staff will clarify when we have a cycle

@ccleeton
Copy link
Collaborator Author

ccleeton commented Dec 6, 2024

Sara and I discussed to change to the following text.

This item can be viewed in person at xxxx (library name) during business hours.

We also want to remove "manuscript" word from Mudd name that comes into the cart.

@tpendragon tpendragon added the dls-work-cycle Tickets designated for the next work cycle (limit 16) label Dec 12, 2024
@Seanwarren77
Copy link
Contributor

Screenshot 2024-12-12 at 12 23 49 PM

If I understand correctly, the text in this screen shot that says "view this item in the Mudd Manuscript Library" should be changed to, "This item can be viewed in person at Mudd Library during business hours."

  1. The current link to Mudd Library goes to Special Collections, should that go to the Mudd Library page which is
    https://library.princeton.edu/services/special-collections/seeley-g-mudd-manuscript-library

  2. Should we link business hours to this page: https://library.princeton.edu/hours

@ccleeton
Copy link
Collaborator Author

So we want the common text to say This item can be viewed in person (library name as a hyperlink here) during business hours.

For the library name for Mudd to make it shorter - Mudd Library
For the library name for Firestone stays the same - Firestone Library

we would prefer to keep the link to this as it gives alot of specific information about visiting special colletions
https://library.princeton.edu/services/special-collections/visit-special-collections

@ccleeton
Copy link
Collaborator Author

ccleeton commented Jan 7, 2025

Thanks for the great work on this ticket. I added a screen shot of a few items that I was testing in the cart and it seems that the review note is overriding the location as you can see in the top item in the cart.
I am also not seeing the Mudd Library name change.
Did it maybe not index yet?
cart

@ccleeton ccleeton reopened this Jan 7, 2025
@Seanwarren77
Copy link
Contributor

@ccleeton Sorry about that just deployed, please let me know if this fixes the issue.

@ccleeton
Copy link
Collaborator Author

ccleeton commented Jan 8, 2025

Thanks Sean, I see the change in the Mudd Library but I am still seeing that the review note is overriding the location as you can see in the top item in the cart.
We would prefer to have the review note and the location appear. Is that possible?

@ccleeton ccleeton reopened this Jan 8, 2025
@Seanwarren77
Copy link
Contributor

If we have them both appear at the same time, this is what it will look like:

This item can be viewed in person at Mudd Library. These item(s) will be paged upon your arrival to the reading room. Item may need to be reviewed by staff for access.

The above text seems to provide conflicting information. Users may be upset when they show up to the reading room only to be denied access. How often is access granted when users show up to the reading room requesting restricted materials? If this warrants discussion lets set up a meeting.

@ccleeton
Copy link
Collaborator Author

While I can understand your concern. We would prefer users to have a potential warning. It is always possible that a researcher might not be able to see something.

@sdellis
Copy link
Member

sdellis commented Jan 15, 2025

@ccleeton I'm confused by this response. I think we should schedule a Zoom call to discuss and come to an agreement.

@ccleeton
Copy link
Collaborator Author

Sure, to clarify due to access restrictions set but the University or donors there is always a chance that something can not be viewed in person. So we want to let researchers know physically where the items are on campus but also that there is a chance based on a restriction they might not be able to see it in person.

@sdellis
Copy link
Member

sdellis commented Jan 24, 2025

Item may need to be reviewed by staff for access.

I would suggest more specificity in the above sentence to avoid confusion and potentially unhappy patrons:

Staff will review the items upon your arrival to determine access.

@arusinko
Copy link

Are the restrictions that would prompt this message in the shopping cart pulled from the component level?

@sdellis
Copy link
Member

sdellis commented Feb 7, 2025

We should verify that the item-level restriction is what the Request Cart is basing the notice on.

The following collection has an access restriction at the item level, however all components get the request cart restriction note:
https://findingaids.princeton.edu/catalog/C0977
https://findingaids.princeton.edu/catalog/C0977_c0355

Similar collection: https://findingaids.princeton.edu/catalog/C0108

Both restricted and unrestricted items have the same text for "review" and "restricted" type access notes. Is this what we want? @ccleeton will check with Sara.

@ccleeton
Copy link
Collaborator Author

With a conversation with Sara and public services staff, we have determined that our restrictions across the repositories make it to challenging to clarify to users in the shopping cart. We want to leave the location and paging info and remove the access restrictions text.
We appreciate the work on this but it became much too complicated to manage in this way.

I think once the restrictions are removed we can close this ticket.

@sdellis
Copy link
Member

sdellis commented Feb 20, 2025

@bess and @Seanwarren77
This is the commit that we should roll back to close this ticket:
d86c207

It allows the for the finding aids component's restricted parameter to be sent to the RequestCart (Vue.js component) for rendering the note.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dls-work-cycle Tickets designated for the next work cycle (limit 16) On Deck Next issues marked by product owners for working on. user experience
Projects
None yet
Development

No branches or pull requests

7 participants