Skip to content

Fix rate limiting docs flow layout#6484

Merged
chronark merged 1 commit into
mainfrom
eng-2925-fix-cropped-ascii-diagram-in-rate-limiting-docs-c889
Jun 19, 2026
Merged

Fix rate limiting docs flow layout#6484
chronark merged 1 commit into
mainfrom
eng-2925-fix-cropped-ascii-diagram-in-rate-limiting-docs-c889

Conversation

@chronark

@chronark chronark commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Important

Unkey is not accepting external pull requests at this time. Pull requests from people outside the Unkey team will not be reviewed or merged.

What does this PR do?

Replaces the cropped ASCII flow diagram on the rate limiting "How it works" page with the existing Mintlify Steps component. The content is the same regional convergence flow, but it no longer depends on a fixed-width code block.

The Linear issue is linked by the branch name.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • Enhancement (small improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How should this be tested?

  • git diff --check
  • npx --yes mintlify@4.2.573 validate from docs/product

mise wasn't available in this checkout, so Mintlify validation was run directly with npx. The command emitted dependency deprecation warnings before validation passed.

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Read Internal Workflow Guide
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand areas
  • Ran pnpm build
  • Ran pnpm fmt
  • Ran mise run fmt
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the Unkey Docs if changes were necessary

Co-authored-by: linear-code[bot] <222613912+linear-code[bot]@users.noreply.github.com>
@linear-code

linear-code Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

ENG-2925

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
dashboard Ready Ready Preview, Comment Jun 17, 2026 6:23pm
design Ready Ready Preview, Comment Jun 17, 2026 6:23pm

Request Review

@mintlify

mintlify Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
unkey 🟢 Ready View Preview Jun 17, 2026, 6:23 PM

@pullfrog

pullfrog Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

This run was cancelled 🛑

The workflow was cancelled before completion. Please check the link below for details.

Pullfrog  | View workflow run | via Pullfrog𝕏

@chronark chronark merged commit bdbcb38 into main Jun 19, 2026
17 checks passed
@chronark chronark deleted the eng-2925-fix-cropped-ascii-diagram-in-rate-limiting-docs-c889 branch June 19, 2026 07: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.

1 participant