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

Items not clearing from the Selected Items list after the end of lease date #1701

Closed
Karthikeya212 opened this issue Mar 10, 2017 · 3 comments

Comments

Projects
None yet
4 participants
@Karthikeya212
Copy link
Collaborator

commented Mar 10, 2017

Descriptive summary

Server: spruce.dlib.indiana.edu
Testing: Regression
Browser: Chrome(56.0.2924.87 (64-bit))
Operating System: Windows10
Issue: After the end of the lease date of access for a user, the items are not getting removed from the selected items list.

Expected behavior

Actual behavior

Steps to reproduce the behavior

  1. Login to spruce system with at least manager level user.
  2. Create a new item and set special access to an Avalon user who is not a member of this collection.
  3. Login to spruce using the Avalon user details.
  4. Search for the item that he has special access(lease) to.
  5. Select the item and keep it like that (selection may include other items as well).
  6. Wait for the lease to be completed (based on the end date you provide).
  7. After the lease is over, check the selected items list. The count still shows same(it is not updated), but the item is not present in the list.

Related work

Link to related tickets or prior related work here.

@Karthikeya212

This comment has been minimized.

Copy link
Collaborator Author

commented Mar 10, 2017

Linked to issue #1688

@lealeelu lealeelu self-assigned this Jun 8, 2017

@lealeelu

This comment has been minimized.

Copy link
Contributor

commented Jun 8, 2017

Looking at the problem, it looks like it would be best to make a rake task and add that task to a daily scheduled job in config/schedule.rb. The rake task would search for outdated bookmarks and then use a new class that would delete the bookmark. mo.remove_bookmark is on the MO and might not be best.

@davidschober

This comment has been minimized.

Copy link

commented May 17, 2019

edge case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.