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

Remove object_thumbnail? method #1749

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Remove object_thumbnail? method #1749

merged 1 commit into from
Nov 17, 2023

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Nov 15, 2023

SDR never makes metadata with the thumb attribute anymore. This method would return false for every object

There were only 7 remaining objects that had this resource type. Andrew remediatied these:

druid:pm026jg3348
druid:gb724cq8086
druid:cb810hh5010
druid:dm864vr8555
druid:gs516wd2597
druid:xs076vf8779
druid:ty839db5658

@jcoyne jcoyne marked this pull request as draft November 15, 2023 13:42
@jcoyne jcoyne force-pushed the remove-object-thumbnail branch 3 times, most recently from 2cdaefe to 0896da6 Compare November 16, 2023 21:29
@jcoyne jcoyne marked this pull request as ready for review November 17, 2023 02:56
@jcoyne jcoyne force-pushed the remove-object-thumbnail branch from 0896da6 to 6e3f680 Compare November 17, 2023 15:49
SDR never makes metadata with the thumb attribute anymore. This method would return false for every object
@jcoyne jcoyne force-pushed the remove-object-thumbnail branch from 6e3f680 to 0e8cb3e Compare November 17, 2023 15:55
@mjgiarlo mjgiarlo merged commit bbc149d into main Nov 17, 2023
1 check passed
@mjgiarlo mjgiarlo deleted the remove-object-thumbnail branch November 17, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants