-
Notifications
You must be signed in to change notification settings - Fork 300
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
DOI Badge Is Broken #743
Comments
@fedorkotov I wasn't paying attention and didn't get that change committed with the other PR. I've fixed it and CI is running on the merge right now. If it's not fixed after CI has finished, I'll edit directly on GitHub so that I can use the preview. |
So the link is updated in the badge now, but it seems we're not alone in having the badge refuse to display. There are a couple of odd things in their generated SVG file. I may experiment with that a little bit. |
It's taking about that long for the image to load for me in the US too. Maybe GitHub is timing out while trying to load the image? I tried uploading the SVG to my own web server and it loads fine in the readme. I guess I'll change the readme to point to the temporary image location and leave this issue open so we can circle back when Zenodo has their issue fixed. |
Ok, change made and badge is visible again. Thanks for the feedback on this issue @fedorkotov |
Looks like there might be a content header issue on Zenodo's side in addition to the long load times. |
This appears to be fixed now on Zenodo's side, although there has been no communication on what happened. The issue was closed and the badge SVG now works. |
Thanks for looking into this @jmwright ! |
I noticed that the DOI badge in the readme was just displaying text instead of an image, and looked into it. It looks like the links for the badge SVG and the URL the badge points to have changed.
Old DOI Badge Markdown:
New DOI Badge Markdown:
I clicked on the DOI badge in the right sidebar here to get the new information. I can change it, but wasn't involved in adding the DOI badge and I wanted to make sure I wasn't missing something.
The text was updated successfully, but these errors were encountered: