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

Fix acknowledgement logos in readme #427

Merged
merged 7 commits into from
Mar 7, 2024

Conversation

efaulhaber
Copy link
Member

No description provided.

@efaulhaber efaulhaber requested a review from svchb March 5, 2024 12:51
@efaulhaber efaulhaber added this to the 0.1 milestone Mar 5, 2024
@efaulhaber
Copy link
Member Author

For some reason, the links stop working after some time (minutes/hours).
Interestingly, we get private-user-images.githubusercontent.com links, whereas in trixi-framework/Trixi.jl#1390, they get user-images.githubusercontent.com links. Note that I specifically created a new issue after we made the repo public.

@efaulhaber efaulhaber marked this pull request as draft March 5, 2024 15:23
@efaulhaber
Copy link
Member Author

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.

@sloede
Copy link
Member

sloede commented Mar 5, 2024

For some reason, the links stop working after some time (minutes/hours). Interestingly, we get private-user-images.githubusercontent.com links, whereas in trixi-framework/Trixi.jl#1390, they get user-images.githubusercontent.com links. Note that I specifically created a new issue after we made the repo public.

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

@efaulhaber
Copy link
Member Author

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 private-user-images.githubusercontent.com, but you guys didn't use that link, but posted the link that is added by GitHub in markdown when you paste the image. Interestingly, these are not the same:
grafik
grafik
Apparently, the private-user-images.githubusercontent.com URLs are only temporary.

@sloede
Copy link
Member

sloede commented Mar 6, 2024

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.

@efaulhaber efaulhaber marked this pull request as ready for review March 6, 2024 07:11
Comment on lines +3 to +4
[![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)
Copy link
Member Author

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.

Comment on lines +5 to +6
[![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)
Copy link
Member Author

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.

@svchb svchb enabled auto-merge (squash) March 6, 2024 23:56
@efaulhaber efaulhaber disabled auto-merge March 7, 2024 17:29
@efaulhaber efaulhaber merged commit 84b914a into trixi-framework:main Mar 7, 2024
3 checks passed
@efaulhaber efaulhaber deleted the fix-logos-readme2 branch March 7, 2024 17:29
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.

4 participants