-
Notifications
You must be signed in to change notification settings - Fork 51
Science Museum: Handle unrecognized licenses #850
Conversation
Based on the high urgency of this PR, the following reviewers are being gently reminded to review this PR: @obulat Excluding weekend1 days, this PR was updated 2 day(s) ago. PRs labelled with high urgency are expected to be reviewed within 2 weekday(s)2. @AetherUnbound, if this PR is not ready for a review, please draft it to prevent reviewers from getting further unnecessary pings. Footnotes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried running the script this way and checking the items without the license. They are either not openly-licensed, or not images. Great tests!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was able to reproduce the error in main
(quite fast!) as well, and the fix looks good 👍
@@ -25,6 +25,7 @@ | |||
(1940, 1965), | |||
(1965, 1990), | |||
(1990, 2020), | |||
(2020, 2023), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice observation and addition! 💯
* Science Museum: Handle unrecognized licenses * Add present year range
Fixes
Fixes WordPress/openverse#1580 by @AetherUnbound
Description
This PR addresses an issue wherein licenses which were expected to follow the CC license pattern actually ended up being generic license types. I have changed the tests to use in-line data, and added all of the cases I was able to capture that caused issues when I ran this locally.
I also added a new year range, 2020-2023, so we can include data from the last two years 😄
Testing Instructions
just test
If you want to observe the failure:
main
(0, 1500)
(this was the one that originally caused the error)Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin