-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Docusaurus build fix - Sidebar and missing image issues #1243
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -142,7 +142,7 @@ CIS workflows integrate with: | |||||
|
|
||||||
| ## Related Documentation | ||||||
|
|
||||||
| - **[BMM Documentation](../../bmm/docs/index.md)** - Core BMad Method documentation | ||||||
| - **[BMM Documentation](../bmm/index.md)** - Core BMad Method documentation | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Incorrect path: link points to ../bmm/ but folder is bmm-bmad-method/. Same issue as in the BMGD index file. The BMM module folder is now Current link: 🔎 Fix the documentation link path
Suggested change
🤖 Prompt for AI Agents
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These issues were noted during build, and since the docs are going to be moved completely to new directories, the focus on this PR was to get the build flow working again. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @forcetrainer, understood! Makes sense to focus on getting the build working first. Since you're planning a broader docs reorganization, it's fine to address these path inconsistencies as part of that future work. Thanks for the context! 👍 |
||||||
|
|
||||||
| --- | ||||||
|
|
||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Incorrect path: link points to ../bmm/ but folder is bmm-bmad-method/.
The PR migrates the BMM module from
modules/bmm/tomodules/bmm-bmad-method/, but this link still uses the old short name.Current link:
../bmm/index.mdExpected path:
../bmm-bmad-method/index.mdThis will result in a broken link (404) when the docs are built.
🔎 Fix the documentation link path
🤖 Prompt for AI Agents