Skip to content

docs: remove multi-input API references#283

Merged
arcticfloyd1984 merged 2 commits intomainfrom
yash/remove-multi-input-docs
Mar 27, 2026
Merged

docs: remove multi-input API references#283
arcticfloyd1984 merged 2 commits intomainfrom
yash/remove-multi-input-docs

Conversation

@arcticfloyd1984
Copy link
Copy Markdown
Collaborator

Part of the multi-input API deprecation project. Removes:

  • The swap-multi-input API reference page
  • Multi-input navigation entry from docs.json
  • Multi-input section from chain-abstraction use case
  • Multi-input mention from app-fees warning
  • Multi-input link from API overview

Part of the multi-input API deprecation project. Removes:
- The swap-multi-input API reference page
- Multi-input navigation entry from docs.json
- Multi-input section from chain-abstraction use case
- Multi-input mention from app-fees warning
- Multi-input link from API overview

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mintlify
Copy link
Copy Markdown
Contributor

mintlify bot commented Mar 26, 2026

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

Project Status Preview Updated (UTC)
unevenlabs 🟢 Ready View Preview Mar 26, 2026, 9:18 PM

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

Claude finished @arcticfloyd1984's task in 41s —— View job


Code Review: Remove Multi-Input API References

  • Review PR diff and changed files
  • Check for any remaining multi-input references
  • Post review feedback

Overall: Looks good! Clean removal of multi-input references across the docs. One thing to flag:


Remaining multi-input reference

There's still a "Multi-input" entry in the supported transaction types table at features/app-fees.mdx:47:

| Multi-input | ❌ |

Since the multi-input API is being deprecated and removed from docs, this table row should likely be removed as well to avoid referencing a feature that no longer exists in the documentation.

Fix this →

Minor note on references/api/overview.mdx

After removing the multi-input link, the "Advanced" section (overview.mdx:37-39) now only contains a single entry (Fast Fill). Not necessarily an issue, but worth noting — if more items get removed in the future, consider whether the "Advanced" heading is still warranted.


Everything else is clean — nav entry removed from docs.json, API page deleted, warning text updated, and chain-abstraction section removed. No broken links detected from the changes.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

Claude finished @arcticfloyd1984's task in 49s —— View job


Code Review: Remove Multi-Input API References

  • Review PR diff and changed files
  • Check for remaining multi-input references
  • Post review feedback

Overall: Looks good! Clean removal of all multi-input references. The latest commit (68f7ba3) addressed the previously flagged table row in app-fees.mdx — no remaining multi-input references found in the codebase.


Minor note on references/api/overview.mdx

After the removal, the "Advanced" section (lines 37-39) now contains only a single entry (Fast Fill). Not a blocker, but worth noting — if more items are removed in the future, consider whether the heading is still warranted or if the entry could be folded into the section above.

Newline fix in features/app-fees.mdx

The diff also adds a trailing newline to the end of app-fees.mdx (previously missing). Good housekeeping.


No issues found — LGTM.

@arcticfloyd1984 arcticfloyd1984 merged commit 7b28ca7 into main Mar 27, 2026
4 checks passed
@arcticfloyd1984 arcticfloyd1984 deleted the yash/remove-multi-input-docs branch March 27, 2026 13:21
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