Skip to content

Update relay fees doc#284

Merged
pedromcunha merged 1 commit intomainfrom
feature/relay-fees-update
Mar 27, 2026
Merged

Update relay fees doc#284
pedromcunha merged 1 commit intomainfrom
feature/relay-fees-update

Conversation

@pedromcunha
Copy link
Copy Markdown
Member

No description provided.

@pedromcunha pedromcunha requested a review from ted-palmer March 27, 2026 13:37
@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, 1:37 PM

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026

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


Code Review

  • Gather context and read changed files
  • Review changes against documentation standards
  • Post review feedback

Overall this is a solid update — the new volume tier table, fee category definitions, expanded price impact descriptions, and Fee Sponsorship section are all clear and well-structured. A few minor items:

1. Inconsistent casing: "USDs" vs "USDS" (fees.mdx:32)

Line 31 uses USDS (uppercase), but line 32 uses USDs (mixed case):

- **Stablecoin swaps**: ...USDe and USDS.
- **Major token swaps**: ...USDe and USDs).

Should be consistent — likely USDS in both places.

Fix this →

2. Missing newline at end of file (fees.mdx:84)

The file doesn't end with a newline character. Most linters and editors expect a trailing newline.

3. Minor: verify "Aster" token name (fees.mdx:33)

e.g. AVAX on Avalanche to Aster on Binance Smart Chain

Just double-checking — is "Aster" the intended token name here? If referring to the Astar Network token, the ticker is typically ASTR.


Everything else looks good:

  • Volume tier table is clean and easy to scan
  • Fee category definitions are clear with helpful examples
  • expandedPriceImpact field name fix (was expanded price impact) is correct
  • New sponsored field and <Tip> callout for display mapping are useful additions
  • New Fee Sponsorship section ties in well with the existing docs
  • Follows the repo's documentation standards (active voice, heading hierarchy, callout usage)

@pedromcunha pedromcunha merged commit 2f7fe3b into main Mar 27, 2026
4 checks passed
@pedromcunha pedromcunha deleted the feature/relay-fees-update branch March 27, 2026 13:38
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