-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix acknowledgement logos in readme #427
Conversation
For some reason, the links stop working after some time (minutes/hours). |
Of course, we can just use the Trixi issue to host our images as well, but we also want to host images for the docs, so it would be nice if we could fix that problem here. |
Seems fixed in #438, or not? I now get something like `https://github-production-user-asset-6210df.s3.amazonaws.com/44124897/310171141-05132bf1-180f-4228-b30a-37dfb6e36ed5.png?``` for the hereon logo, for example |
Okay, I think I know what's going on now. I think GitHub changed the way they store assets. I was looking at the first posts in trixi-framework/Trixi.jl#1390, where the posted link is the same as the "copy image address" URL. In the later posts, that changed to |
Ah yes, I didn't remember to tell you that I played around with this before and found that each time the picture URL is accessed, it is actually resolved to something else. You need to use the asset URL that is stored in Markdown, not the (temporary) resolved URL when linking to that image. |
[![Docs-stable](https://img.shields.io/badge/docs-stable-blue.svg)](https://trixi-framework.github.io/TrixiParticles.jl/stable) | ||
[![Docs-dev](https://img.shields.io/badge/docs-dev-blue.svg)](https://trixi-framework.github.io/TrixiParticles.jl/dev) |
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 also added the docs badges. Note that the first one doesn't work yet, but will (hopefully) work in 2 days when the package is registered.
[![Slack](https://img.shields.io/badge/chat-slack-e01e5a)](https://join.slack.com/t/trixi-framework/shared_invite/zt-sgkc6ppw-6OXJqZAD5SPjBYqLd8MU~g) | ||
[![Youtube](https://img.shields.io/youtube/channel/views/UCpd92vU2HjjTPup-AIN0pkg?style=social)](https://www.youtube.com/@trixi-framework) |
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.
And I moved the social badges in front of the CI.
9ea55d6
to
0884ace
Compare
No description provided.