Skip to content

Fix: Convert Upstream Issue and LFX URLs into clickable hyperlinks#7563

Open
piyush2693 wants to merge 6 commits intolayer5io:masterfrom
piyush2693:fix/lfx-clickable-links
Open

Fix: Convert Upstream Issue and LFX URLs into clickable hyperlinks#7563
piyush2693 wants to merge 6 commits intolayer5io:masterfrom
piyush2693:fix/lfx-clickable-links

Conversation

@piyush2693
Copy link
Copy Markdown

@piyush2693 piyush2693 commented Mar 26, 2026 •
edited
Loading

Description

This PR fixes the issue where Upstream Issue and LFX URLs were displayed as plain text instead of clickable hyperlinks across LFX mentorship sections.

Fixes: #7554

Changes

  • Converted raw URLs into proper Markdown clickable links
  • Ensured consistency across all LFX mentorship sections (2022,2023, 2024 and 2025)
  • Improved formatting and readability of links

Result

  • All URLs are now clickable and properly formatted
  • Improved user experience and navigation across mentorship pages
  • Consistent link formatting across all years

Notes

  • Verified changes using Netlify preview
  • No breaking changes introduced

@piyush2693 piyush2693 force-pushed the fix/lfx-clickable-links branch from 15b6a2a to 459c5ce Compare March 26, 2026 11:47
@piyush2693 piyush2693 mentioned this pull request Mar 26, 2026
Open
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 26, 2026

🚀 Preview for commit 459c5ce at: https://69c52052c6c34a2de6796543--layer5.netlify.app

@piyush2693
Copy link
Copy Markdown
Author

piyush2693 commented Mar 26, 2026

Hi @l5io
I’ve reviewed at https://69c52052c6c34a2de6796543--layer5.netlify.app/ and verified that the Upstream Issue and LFX URLs are now clickable and working as expected.
Please review the PR when convenient.
Thanks!

Bhumikagarggg
Bhumikagarggg suggested changes Mar 26, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@Bhumikagarggg Bhumikagarggg left a comment

Choose a reason for hiding this comment

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

Image

there is still URL which is not clickable in lfx-2024

Bhumikagarggg
Bhumikagarggg suggested changes Mar 26, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@Bhumikagarggg Bhumikagarggg left a comment

Choose a reason for hiding this comment

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

Screenshot 2026-03-26 at 7 33 57 PM

this is in lfx-2023

src/collections/programs/lfx-2023/lfx-2023.mdx Outdated
- Mentor(s): <Link to="/community/members/lee-calcote">Lee Calcote</Link>
- Upstream Issue (URL): https://github.com/meshery/meshery/issues/6447
- LFX URL: https://mentorship.lfx.linuxfoundation.org/project/bb8ddf84-31d7-4a89-9e4b-e6aa9601c0db
- Upstream Issue (URL): [ttps://github.com/meshery/meshery/issues/6447](https://github.com/meshery/meshery/issues/6447)
Copy link
Copy Markdown
Contributor

@Bhumikagarggg Bhumikagarggg Mar 26, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

H is missing

piyush2693 added a commit to piyush2693/layer5 that referenced this pull request Mar 26, 2026
@piyush2693
Fix-some changes on previous PR layer5io#7563
fd13564
@piyush2693 piyush2693 force-pushed the fix/lfx-clickable-links branch 2 times, most recently from def5ed4 to 6484229 Compare March 26, 2026 14:45
@piyush2693
Fix: Convert Upstream Issue and LFX URLs into clickable hyperlinks
36fe678 
Signed-off-by: Piyush <pk9995114@gmail.com>
@piyush2693
Fix-some changes on previous PR layer5io#7563
3c4b04e 
Signed-off-by: Piyush <pk9995114@gmail.com>
@piyush2693
Fix: Convert remaining non-clickable URL in LFX 2024 section
2d8997b 
Signed-off-by: Piyush <pk9995114@gmail.com>
@piyush2693 piyush2693 force-pushed the fix/lfx-clickable-links branch from 6484229 to 2d8997b Compare March 26, 2026 14:49
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 26, 2026

🚀 Preview for commit 2d8997b at: https://69c54af579e1f448e2b0af64--layer5.netlify.app

@piyush2693
Copy link
Copy Markdown
Author

piyush2693 commented Mar 26, 2026

hi @Bhumikagarggg ,
now all the links are clickable , please review the PR when convenient.

@Bhumikagarggg
Copy link
Copy Markdown
Contributor

Bhumikagarggg commented Mar 26, 2026

Please edit description.

@piyush2693
Copy link
Copy Markdown
Author

piyush2693 commented Mar 26, 2026

Hi @Bhumikagarggg,

I've updated the PR description with more detailed information. Please review when convenient.

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 26, 2026

🚀 Preview for commit b411045 at: https://69c59d072e08d3046bd84097--layer5.netlify.app

@piyush2693
Delete package-lock.json
036a684 
Signed-off-by: Piyush <150925139+piyush2693@users.noreply.github.com>
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 28, 2026

🚀 Preview for commit 036a684 at: https://69c734bca1cd03650f40200d--layer5.netlify.app

@piyush2693
Remove @babel/runtime dependency
90797e0 
Removed @babel/runtime dependency from package.json.

Signed-off-by: Piyush <150925139+piyush2693@users.noreply.github.com>
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 28, 2026

🚀 Preview for commit 90797e0 at: https://69c7a3b9e922d94c7112bb96--layer5.netlify.app

Bhumikagarggg
Bhumikagarggg suggested changes Mar 28, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@Bhumikagarggg Bhumikagarggg left a comment

Choose a reason for hiding this comment

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

Package.json still there

package.json Outdated
"prepare": "husky || true"
},
"dependencies": {
"@babel/runtime": "^7.29.2",
Copy link
Copy Markdown
Contributor

@Bhumikagarggg Bhumikagarggg Mar 26, 2026

Choose a reason for hiding this comment

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

Remove this file

Copy link
Copy Markdown
Contributor

@Bhumikagarggg Bhumikagarggg Mar 26, 2026

Choose a reason for hiding this comment

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

remove package-lock.json also

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Bhumikagarggg Bhumikagarggg Bhumikagarggg requested changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

area/careers

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix non-clickable links in issues section

3 participants

@piyush2693 @l5io @Bhumikagarggg