Skip to content

docs: clarify navigation and Solana limit orders - #675

Open
igorroncevic wants to merge 3 commits into
mainfrom
docs/integration-navigation
Open

igorroncevic wants to merge 3 commits into
mainfrom
docs/integration-navigation

Conversation

@igorroncevic

@igorroncevic igorroncevic commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Description

Clarify the integration navigation and explain how to create Solana limit orders without a quote.

Changes

  • Put the integration overview first and rename the Solana entry to "Orders on Solana".
  • Add a user-funded limit-order example with partial fills and expiry.
  • Clarify when a quote is needed and how transaction expiry differs from order expiry.

@igorroncevic
igorroncevic requested a review from harisang October 23, 2026 09:39
@igorroncevic
igorroncevic requested a review from a team as a code owner October 23, 2026 09:39
@vercel

vercel Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Oct 29, 2026 4:36pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 11e7fe03-1ab6-4ab0-b6b4-c7a33ab2671d
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@harisang

Copy link
Copy Markdown
Contributor

I think Solana should be at the very end. Also, could this be a subsection of the "SDK Integration" section?

@igorroncevic

Copy link
Copy Markdown
Contributor Author

I think Solana should be at the very end. Also, could this be a subsection of the "SDK Integration" section?

It's not really just an SDK thing, it's SDK section is just part of the page, and we have concept explanations that are not relevant to the SDK per-se.

@igorroncevic igorroncevic changed the title docs: clarify integration navigation docs: clarify navigation and Solana limit orders Oct 29, 2026
@harisang

harisang commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

I still think Orders on Solana is misplaced, in that the ordering on the left doesn't make much sense; all other subsections are network-independent and Solana appears second with a somewhat vogue title.

Also the title on the left doesn't much the title of the section

image

This branch was successfully deployed

1 active deployment
Preview — b2a13ce3 Deployed Oct 29, 2026 by vercel[bot]
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.

3 participants