Skip to content

Fix Bootstrap user-guide image links - #188

Merged
dborovcanin merged 1 commit into
mainfrom
fix-img-links
Sep 2, 2026
Merged

dborovcanin merged 1 commit into
mainfrom
fix-img-links

Conversation

@ianmuchyri

Copy link
Copy Markdown
Contributor

Summary

  • content/docs/user-guide/bootstrap.mdx used ../../img/bootstrap/... (one ../ too many) for its 6 screenshots, while sibling pages at the same depth (e.g. message-views.mdx) correctly use ../img/....
  • The extra ../ made lib/remark-doc-images.ts emit a URL with a literal .. segment baked in, which failed mdx-components.tsx's DOC_IMAGE_PATTERN check and fell through to assetPath(), double-prepending the base path. End result was a broken /docs/magistrala/docs/img/bootstrap/... URL instead of /docs/magistrala/img/bootstrap/....
  • Fixed by dropping the extra ../ on all 6 image references. The images themselves are already correctly uploaded to R2 under magistrala-docs/img/bootstrap/*.

Test plan

  • Verify build/deploy renders the 6 Bootstrap screenshots correctly on the live user-guide page

Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
magistrala-docs 4b67cb0 Commit Preview URL

Branch Preview URL
Sep 02 2026, 07:08 AM

@dborovcanin
dborovcanin merged commit 27de55f into main Sep 2, 2026
2 checks passed
@dborovcanin
dborovcanin deleted the fix-img-links branch September 2, 2026 08:23
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