Turn off bookmarks for guest users?

[mikeapp]

Story

Guest users can bookmark items. We have also had problems on other systems with guest user accounts filling up the Users table. I suggest that we turn off guest users if possible (see https://groups.google.com/g/blacklight-development/c/2_cg97j97YE )

Acceptance

• [x] Turn off guest user functionality
• [x] When clicking on Bookmarks link, users that are not logged in get a "you need to log in" message
• [x] Bookmark checkboxes don't display for non-logged-in users

[jpengst]

PR: https://github.com/yalelibrary/yul-dc-blacklight/pull/922

[jpengst]

@mikeapp I think removing the devise-guests gem is a bit more complicated than described in the thread above. There's a bunch of references to token/current/guest users throughout blacklight that gets lost when the guests gem is removed.
As a temporary workaround to fulfill the ticket, I can keep the devise-guests gem and then limit the bookmark features to signed in users only. But I think removing the devise-guests gem would require a separate ticket. Thoughts on this?

Rob mentioned that guests can be auto-removed and we can set that job to run as often as we like.

[jpengst]

PR: https://github.com/yalelibrary/yul-dc-blacklight/pull/1019

[K8Sewell]

Deployed to Demo with commit

[sshetenhelm]

• Confirmed that bookmark boxes are not visible for users in search results when not logged in
• Confirmed that bookmark boxes are not visible for users on parent object pages when not logged in
• Confirmed that users who are not logged in receive a "please log in" message when clicking "Saved Items" at the top of the page
• Confirmed that bookmark functionality continues to work as expected for users after logging in

On feature branch, so ready to close. Thank you!