Skip to content

Conversation

@kishore08-07
Copy link
Contributor

@kishore08-07 kishore08-07 commented Dec 20, 2025

Description
This PR fixes broken links and navigation issues in the learning paths section by correcting path generation and template literal syntax.

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Copilot AI review requested due to automatic review settings December 20, 2025 18:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes broken links and navigation issues in the learning paths section by correcting path generation and template literal syntax.

Key changes:

  • Fixed 404 errors when course links are opened directly by ensuring absolute paths (with leading /)
  • Corrected template literal syntax in Course-Overview component to properly generate dynamic URLs
  • Standardized path generation across learning path components using proper field references

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/sections/Learn/LearnPage-Sections/learning-path.js Added leading slash to learning path URLs to create absolute paths preventing 404 errors
src/sections/Learn-Layer5/index.js Changed to use slug field with absolute path for consistent navigation
src/sections/Learn-Layer5/Course-Overview/index.js Fixed template literal syntax (changed from double quotes to backticks) for dynamic URL generation in both the back button link and "Start Again" button

@l5io
Copy link
Contributor

l5io commented Dec 20, 2025

🚀 Preview for commit 62199ae at: https://6946f3a8964a49225f11eaec--layer5.netlify.app

@saurabhraghuvanshii saurabhraghuvanshii merged commit 93755cc into layer5io:master Dec 20, 2025
10 checks passed
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.

3 participants