Skip to content

docs: escape quote currency values#528

Merged
DhruvPareek merged 1 commit into
mainfrom
dp/docs-fix-quote-currency-rendering
May 28, 2026
Merged

docs: escape quote currency values#528
DhruvPareek merged 1 commit into
mainfrom
dp/docs-fix-quote-currency-rendering

Conversation

@DhruvPareek
Copy link
Copy Markdown
Contributor

@DhruvPareek DhruvPareek commented May 28, 2026

Current docs formatting on this line is messed up: Screenshot 2026-05-28 at 3.06.32 PM.png

Now it looks like:
Screenshot 2026-05-28 at 3.07.21 PM.png

@vercel
Copy link
Copy Markdown

vercel Bot commented May 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
grid-flow-builder Ready Ready Preview, Comment May 28, 2026 10:07pm

Request Review

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 28, 2026

Greptile Summary

This PR fixes two unescaped $ characters in a prose sentence within quote-system.mdx so they render as literal dollar signs rather than being interpreted as MDX/LaTeX math delimiters.

  • Escapes $1,000 USD and $5.00 on line 105 of the quote explanation sentence using \$.

Confidence Score: 5/5

Documentation-only change with a correct and targeted fix; safe to merge.

The change is a two-character escaping fix in a single MDX prose sentence with no logic, API, or schema impact.

No files require special attention, though the same file has additional unescaped $ signs in the Rate Details Breakdown section that may warrant a follow-up.

Important Files Changed

Filename Overview
mintlify/platform-overview/core-concepts/quote-system.mdx Escapes two $ signs in the quote explanation sentence (line 105) to prevent MDX math-delimiter misinterpretation; several other unescaped $ occurrences remain in the same file.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["MDX source: $1,000 / $5.00"] -->|"Before fix"| B["Mintlify parser interprets $ as math delimiter"]
    B --> C["Broken / garbled render in docs"]
    A2["MDX source: \\$1,000 / \\$5.00"] -->|"After fix"| D["Mintlify parser treats $ as literal character"]
    D --> E["Correct render: $1,000 USD and $5.00"]
Loading

Fix All in Claude Code

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
mintlify/platform-overview/core-concepts/quote-system.mdx:333-341
**Unescaped `$` signs in the same file**

This PR escapes `$` on line 105 to prevent MDX/LaTeX math-delimiter interpretation, but the same unescaped pattern appears in several other prose bullet points in the "Rate Details Breakdown" section (lines 333, 336, 339, 340, 341). If the rendering fix was needed at line 105, these instances are likely affected by the same issue and should also use `\$`.

Reviews (2): Last reviewed commit: "docs: escape quote currency values" | Re-trigger Greptile

@DhruvPareek DhruvPareek marked this pull request as draft May 28, 2026 22:02
@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented May 28, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Grid 🟢 Ready View Preview May 28, 2026, 10:13 PM

@DhruvPareek DhruvPareek merged commit aeaabc0 into main May 28, 2026
9 checks passed
@DhruvPareek DhruvPareek deleted the dp/docs-fix-quote-currency-rendering branch May 28, 2026 22:27
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