Skip to content

fix-2025-07-17-what-we-learned-from-the-chaoss-data-science-hackathon.md #23

Open
rajvi0106 wants to merge 2 commits into
chaoss:mainfrom
rajvi0106:cleanup-wordepress-markup
Open

fix-2025-07-17-what-we-learned-from-the-chaoss-data-science-hackathon.md #23
rajvi0106 wants to merge 2 commits into
chaoss:mainfrom
rajvi0106:cleanup-wordepress-markup

Conversation

@rajvi0106
Copy link
Copy Markdown

fixes/part of #2
Fixes the broken image markup on the what-we-learned-from-the-chaoss-data-science-hackathon blog post as part of #2.

Signed-off-by: rajvi0106 <rajvisinghrathore001@gmail.com>
@rajvi0106 rajvi0106 force-pushed the cleanup-wordepress-markup branch from 9ce0f9e to ec51987 Compare May 11, 2026 18:36
Copy link
Copy Markdown
Contributor

@MoralCode MoralCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing this

I noticed some extra things when comparing the two sites:

  1. the image on the main site (https://chaoss.community/what-we-learned-from-the-chaoss-data-science-hackathon/) has alt text specified when the version on my local build doesn't. Can you make sure to copy the alt text over as well?
  2. I'm noticing that the built page also seems to include a lot of markdown in the page itself. I know you didnt touch these lines of code, but I'm curious if theres something causing this page to not show up correctly (happy to address this in a new issue if you prefer)
Image
  1. I think it should be possible to embed this image using markdown syntax ![alt text here](link to image) - could you try that and see if that works? then we can avoid confusing newcomers with a mix of markdown and HTML syntax if possible

Thanks so much for your contribution!

Comment on lines +20 to +21
<img src="https://chaoss.community/wp-content/uploads/2025/07/leif-christoph-gottwald-iM8dxccK1sY-unsplash-2048x1152.jpg">
<br>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this image exists in this git repo already, could you update this URL to refer to it, so that the site can still build and be viewed offline?

Suggested change
<img src="https://chaoss.community/wp-content/uploads/2025/07/leif-christoph-gottwald-iM8dxccK1sY-unsplash-2048x1152.jpg">
<br>
<img src="{{ "wp-content/uploads/2025/07/leif-christoph-gottwald-iM8dxccK1sY-unsplash-2048x1152.jpg" | relURL }}">
<br>

@rajvi0106
Copy link
Copy Markdown
Author

Thank you for the feedback!

  1. I have added the alt text.
  2. I found the issue with the raw markdown links -
    the reference links at the bottom are using
    {{ baseURL }} which doesn't get processed in
    markdown files. I can fix these by replacing
    {{ baseURL }} with / ,should I include this
    fix in the current PR or open a separate one?
  3. I've updated the image to use markdown syntax
    with the relative path.

@MoralCode
Copy link
Copy Markdown
Contributor

2. I found the issue with the raw markdown links -
the reference links at the bottom are using
{{ baseURL }} which doesn't get processed in
markdown files. I can fix these by replacing
{{ baseURL }} with / ,should I include this
fix in the current PR or open a separate one?

i guess do that in a new issue - / is not always a valid substitute for the base url, so it would be nice if we could get this to work.

I think theres also a lot of other pages where I find-and-replaced the url in the markdown links to use the baseURL variable, so we should probbaly find a solution that works for all of them

Signed-off-by: rajvi0106 <rajvisinghrathore001@gmail.com>
@rajvi0106 rajvi0106 requested a review from MoralCode May 14, 2026 16:09
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.

2 participants