use dataset thumbnail if available
New Contributors
Welcome! New contributors should at least glance at CONTRIBUTING.md, especially the section on pull requests where we encourage you to reach out to other developers before you start coding. Also, please note that we measure code coverage and prefer you write unit tests. Pull requests can still be reviewed without tests or completion of the checklist outlined below. Thanks!
Related Issues
- connects to #5641 (edited by @djbrooke)
Pull Request Checklist
- [ ] Unit tests completed
- [ ] Integration tests: None
- [ ] Deployment requirements, SQL updates, Solr updates, etc.: None
- [ ] Documentation completed
- [ ] Merged latest from "develop" branch and resolved conflicts
Coverage remained the same at 19.488% when pulling 624128dbaf209c953d39dd0bb6ab51323fb0086d on QualitativeDataRepository:IQSS-4894-Open_Graph_metadata into ee874a6224af616fd5db20789f0fcb7521238cbf on IQSS:develop.
Can one of the admins verify this patch?
FWIW: I just resolve the conflict, but, with the changes to make the supplied images higher resolution, this could/should wait until one can specify a higher resolution dataset thumbnail as in #5679
Over at https://github.com/IQSS/dataverse/issues/10220#issuecomment-1897046254 mentioned that this PR increases the dataset logo size (that's my understanding anyway), so it may be a solution for this issue:
- #9418
Hi @qqmyers 👋🏼 I was thinking that this being a solution on the JSF it would be better to implement this logic on /api/datasets/:persistentId/thumbnail so this solution could also be available to the SPA?
I found a site that can preview the Open Graph images. For example https://orcascan.com/tools/open-graph-validator?url=https%3A%2F%2Fdataverse.harvard.edu%2Fdataset.xhtml%3FpersistentId%3Ddoi%3A10.7910%2FDVN%2FTJCLKP does show the generic Dataverse logo:
(Note that it does say "invalid: Your page is missing Open Graph tags, learn more.)
I found a site that can preview the Open Graph images.
At least it gives you a nice detail of what is missing 😄
I opened https://github.com/IQSS/dataverse/issues/10586 to keep track of this for the SPA