Skip to content

fix(docs): correct case of consensus diagram image path (404 on case-sensitive hosting)#443

Open
danilaverbena wants to merge 1 commit into
genlayerlabs:mainfrom
danilaverbena:fix/consensus-diagram-image-case
Open

fix(docs): correct case of consensus diagram image path (404 on case-sensitive hosting)#443
danilaverbena wants to merge 1 commit into
genlayerlabs:mainfrom
danilaverbena:fix/consensus-diagram-image-case

Conversation

@danilaverbena

@danilaverbena danilaverbena commented Jul 19, 2026

Copy link
Copy Markdown

Problem

On the Understand GenLayer Protocol page, the Optimistic Democracy consensus diagram is referenced as:

<img src="/studio/Diagram MAIN.jpg" alt="GenLayer Optimistic Democracy Consensus Diagram" width="100%" />

but the committed asset is public/studio/diagram main.jpg (all lowercase).

Because the reference case (Diagram MAIN.jpg) doesn't match the file on disk (diagram main.jpg), the image resolves fine on case-insensitive filesystems (macOS/Windows dev machines) but returns a 404 on case-sensitive hosting (the Linux/Netlify production build). The result is a broken hero diagram on one of the core concept pages.

For reference, the other image on the site is referenced with the correct lowercase path already, e.g. debugging.mdx uses /studio/syntax-error.png.

Fix

Update the src in pages/understand-genlayer-protocol.mdx to match the actual filename:

-<img src="/studio/Diagram MAIN.jpg" alt="GenLayer Optimistic Democracy Consensus Diagram" width="100%" />
+<img src="/studio/diagram main.jpg" alt="GenLayer Optimistic Democracy Consensus Diagram" width="100%" />

One-line, non-breaking change; the asset itself is unchanged.

Testing

  • Verified the referenced file exists at public/studio/diagram main.jpg (lowercase) and that no file named Diagram MAIN.jpg exists in the repo.
  • The corrected path now matches the committed asset exactly, so it will resolve on case-sensitive production hosting.

Summary by CodeRabbit

  • Documentation
    • Updated the consensus diagram shown in the “Optimistic Democracy: How Consensus Works” section.

The Optimistic Democracy consensus diagram on the "Understand GenLayer Protocol" page references /studio/Diagram MAIN.jpg, but the committed asset is public/studio/diagram main.jpg (lowercase). On case-sensitive hosting (e.g. the Netlify/Linux production build) the image returns 404 and the diagram fails to render, even though it works on case-insensitive local machines (macOS/Windows). Corrected the reference to match the actual filename.
@netlify

netlify Bot commented Jul 19, 2026

Copy link
Copy Markdown

Deploy Preview for genlayer-docs ready!

Name Link
🔨 Latest commit 16fb226
🔍 Latest deploy log https://app.netlify.com/projects/genlayer-docs/deploys/6a5ce3774f71ab0008c2797a
😎 Deploy Preview https://deploy-preview-443--genlayer-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 269ed75a-9c0a-43f3-a443-ce7b94032a03

📥 Commits

Reviewing files that changed from the base of the PR and between 5c1c29d and 16fb226.

📒 Files selected for processing (1)
  • pages/understand-genlayer-protocol.mdx

📝 Walkthrough

Walkthrough

The protocol documentation updates the consensus diagram image reference from Diagram MAIN.jpg to diagram main.jpg.

Changes

Documentation Update

Layer / File(s) Summary
Update consensus diagram reference
pages/understand-genlayer-protocol.mdx
The embedded consensus diagram path uses the updated lowercase filename casing.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise, conventional, and accurately describes the lowercase image-path fix for a broken diagram.
Description check ✅ Passed The description clearly explains the problem, fix, and testing, with enough detail despite using custom headings instead of the template's Description section.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant