Skip to content

Fix fee doc table format#285

Merged
pedromcunha merged 1 commit intomainfrom
fix/fee-doc-table-formatting
Mar 27, 2026
Merged

Fix fee doc table format#285
pedromcunha merged 1 commit intomainfrom
fix/fee-doc-table-formatting

Conversation

@pedromcunha
Copy link
Copy Markdown
Member

No description provided.

@pedromcunha pedromcunha requested a review from ted-palmer March 27, 2026 17:32
@mintlify
Copy link
Copy Markdown
Contributor

mintlify bot commented Mar 27, 2026

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

Project Status Preview Updated (UTC)
unevenlabs 🟢 Ready View Preview Mar 27, 2026, 5:32 PM

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026

Claude finished @pedromcunha's task in 29s —— View job


Code Review: Fix fee doc table format

  • Gather context and review changes
  • Provide review feedback

LGTM — the change correctly converts the Markdown table to an HTML <table> for better rendering control in Mintlify. The data values are preserved accurately across all three volume tiers, and the footnote for "Volume Tier" context is cleanly separated.

A couple of minor observations (non-blocking):

  1. Header text changed slightly — "Same token bridging" → "Same Token Bridge" and "Major token swaps" / "Minor token swaps" → "Major Swaps" / "Minor Swaps". These are shorter, which works well for table headers, but note they no longer exactly match the category names defined in the Relay Fee Categories section below (e.g. "Same token bridging", "Major token swaps"). Worth verifying this is intentional or if you'd prefer consistency between the headers and the category list.

  2. className usageclassName="relay-table" is the JSX/React convention (correct for MDX). Just confirming the relay-table class is defined in your styles — if not, it's harmless but unused.

Otherwise, clean change. Ship it!

@pedromcunha pedromcunha merged commit f4cdffc into main Mar 27, 2026
4 checks passed
@pedromcunha pedromcunha deleted the fix/fee-doc-table-formatting branch March 27, 2026 17:37
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