Add Lesson detail pages feature#54
Conversation
❌ PR checks failedOne or more validation checks failed. Please review the workflow logs to see what went wrong. Common issues:
|
❌ PR checks failedOne or more validation checks failed. Please review the workflow logs to see what went wrong. Common issues:
|
❌ PR checks failedOne or more validation checks failed. Please review the workflow logs to see what went wrong. Common issues:
|
✅ All PR checks passed!
The site builds successfully and all validation checks passed. |
ShouzhiWang
left a comment
There was a problem hiding this comment.
Thank you for putting this together. The functionality of having dedicated lesson pages is definitely valuable.
However, we noticed some significant visual issues during testing (e.g., text visibility in the sidebar is extremely low/illegible in some modes, and the layout hierarchy is inconsistent).
Recommendation: We have actually just received a comprehensive new design draft for the entire website from our designer. To avoid you spending time polishing a design that is about to be completely replaced, I would like to suggest pause work on this PR.
|
Hi @gaurirathi, thank you for this work. We have an internal team member (@ShouzhiWang) currently implementing the lesson detail architecture to match our specific backend requirements. To avoid duplication, we are going to close this PR in favor of #63. We'd love your help on issues labeled help wanted instead! We are still staging things for next milestone. |
This PR adds individual lesson detail pages at the
/lessons/[slug]route.Each lesson now has its own page that displays full metadata and provides a clear link to the external lesson resource, instead of navigating directly away from the site.
##What’s included
src/pages/lessons/[slug].astroThis improves navigation and allows learners to view lesson context before leaving the site.
Closes #28