search

pulibrary / orangelight

PUL Blacklight Project (Ruby 3.1.0, Rails 7.1.4)
21 stars 7 forks source link

The maximum bookmark ids that will allow a page to work is 1024 #2889

Open christinach opened 2 years ago

christinach commented 2 years ago

if the bookmarks are more than 1024 the page will error. See: https://github.com/pulibrary/orangelight/issues/2860

carolyncole commented 2 years ago

Error to the user should be You have exceeded the maximum number of bookmarks! You can only save 1024 bookmarks

sandbergja commented 2 years ago

This relies on https://github.com/projectblacklight/blacklight/pull/2666 getting backported and released.

sandbergja commented 2 years ago

It's now been backported and released in 7.25.2.

kevinreiss commented 1 year ago

Jane will take a look and see if this can split into only a ruby only PR that can be merged with BL 7. Javascript should wait until BL8.

kevinreiss commented 1 year ago

Jane tried this without Javascript helpers, didn't work out well.

sandbergja commented 1 year ago

For a Blacklight 7-compatible work-in-progress branch, see https://github.com/pulibrary/orangelight/pull/3363.

caroldh commented 3 months ago

low priority feature, and this might be resolved with blacklight upgrade?

kevinreiss commented 2 months ago

@caroldh and @ellen-aa, we occasionally get feedback from users (almost always Subject Librarians) that they've amassed a huge number of bookmarks. Do either of you have a sense if folks are doing this in the course of their research or they are using this feature for work that could be better served by using something like the reporting features that Alma has for PUL collections?

ellen-aa commented 2 months ago

@kevinreiss, if you have a few names handy, we could check in with the folks who are amassing a lot of bookmarks. I suspect it's research-related, but it would be good to know for sure. And they could consider using a bibliographic management tool like Zotero or other to keep track of these things...