docs(aimlapi): list AI/ML API in the Popular Providers sidebar - #2
Open
Lookoff-AIMLAPI wants to merge 1 commit into
Open
docs(aimlapi): list AI/ML API in the Popular Providers sidebar#2Lookoff-AIMLAPI wants to merge 1 commit into
Lookoff-AIMLAPI wants to merge 1 commit into
Conversation
The integration pages merged earlier are reachable only from `all_providers`, which is a long alphabetical card wall. The sidebar group beside it is what a reader actually browses, and AI/ML API was not in it. Adds the provider page as the first entry of the Python group. The JavaScript group is left alone on purpose: it nests chat, embeddings and tools pages per provider, and this integration has Python pages only — an entry there would be a group with one link and three dead ones. Placement note, so it is not discovered in review: first means ahead of OpenAI. "Popular Providers" is a curated list on LangChain's own docs, and ordering it is the maintainers' call rather than a contributor's, so this position is a fork decision and should be revisited before anything goes upstream. Everything else in the change is uncontroversial — the page exists, it is complete, and it is currently unreachable from the sidebar. `docs.json` still parses; the added path resolves to `src/oss/python/integrations/providers/aimlapi.mdx`.
|
Thanks for opening a docs PR, @Lookoff-AIMLAPI! When it's ready for review, please add the relevant reviewers:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The integration pages merged in #1 are reachable only from
all_providers, which is a long alphabetical card wall. The sidebar group beside it is what a reader actually browses, and AI/ML API was not in it.Adds the provider page as the first entry of the Python
Popular Providersgroup.Two things worth stating rather than leaving to review
The JavaScript group is deliberately untouched. It nests chat, embeddings and tools pages under each provider, and this integration has Python pages only — an entry there would be a group with one working link and three dead ones.
First means ahead of OpenAI.
Popular Providersis a curated list on LangChain's own documentation, and ordering it is the maintainers' call rather than a contributor's. That position is a fork decision and should be revisited before anything goes upstream — the rest of the change is uncontroversial, since the page exists, is complete, and is simply unreachable from the sidebar today.Verification
docs.jsonstill parses as JSON, and the added path resolves tosrc/oss/python/integrations/providers/aimlapi.mdx.