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

Collection thumbnail migrations #214

Closed
2 tasks
Tracked by #238 ...
ShanaLMoore opened this issue May 3, 2024 · 8 comments
Closed
2 tasks
Tracked by #238 ...

Collection thumbnail migrations #214

ShanaLMoore opened this issue May 3, 2024 · 8 comments

Comments

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented May 3, 2024

Summary

We need a rake task to migrate collection thumbnails. This applies to pals and Adventist

Acceptance Criteria

  • Running the rake task will convert an ActiveFedora (AF) Collection into a Collection Resource.
  • Running the rake task should persist any thumbnail the AF Collection had.

Screenshots or Video

Testing Instructions

  • verify a collection with a thumbnail, after editing, still has a thumbnail
  • Dev to run rake task for confirmation that any pre existing collection thumbnails persist

Notes

We made a similar rake task for this but it is untested:

  • This should probably be removed from Hyku and put into pals knapsack
@jillpe
Copy link

jillpe commented Jun 17, 2024

we will be migrating 2 tenants from Hyku Commons to pals knapsack staging

@ShanaLMoore
Copy link
Contributor Author

find a tenant in staging with a collections that have thumbnails

@jillpe
Copy link

jillpe commented Jun 17, 2024

SoftServ QA: ✅

Collection

Before Editing
Image

** After Editing**

Image

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Jun 17, 2024

Hi @jillpe I think demo is an account we use often, so I would test this on one of the client's staging accounts. The collections you find here may already be updated.

I'm also not convinced that edit/save will work. I think we actually need to run the rake task.

@ShanaLMoore
Copy link
Contributor Author

And actually, i think the rake task currently only applies to pals. moving back to dev for further review.

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Jun 27, 2024

Rob is moving prod data into pals staging so this would be a great time to test.

Do we need to add a lazy migration?

@ShanaLMoore ShanaLMoore added the blocked other work must be completed first label Jul 5, 2024
@ShanaLMoore
Copy link
Contributor Author

How to get local pre valkyrie data: https://assaydepot.slack.com/archives/C0311DN2YCA/p1721771295691059

Image

@ShanaLMoore
Copy link
Contributor Author

ref: samvera/hyku#2273 (review)

This is no longer needed and has been removed from Hyku

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants