Skip to content

Brody/doc 60 port integrations docs #153

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 42 commits into from
Aug 22, 2025
Merged

Conversation

beklapko
Copy link
Contributor

@beklapko beklapko commented Aug 13, 2025

Current preview: https://langchain-5e9cc07a-preview-brodyd-1755879478-9f01bc4.mintlify.app/oss/python/integrations/providers/anthropic

Related task: https://linear.app/langchain/issue/DOC-60/port-integrations-docs

What this PR does

What this PR does not do

  • Adds the Integration section of the docs to the nav
  • Fix all the broken links
  • Fix every rendering issue
  • Fix every title
  • Finalize the sidebar
  • Finalize table appearance and other theme settings
  • Set up the contributing page

lnhsingh and others added 30 commits August 4, 2025 15:24
## Summary

Preview link:
https://langchain-5e9cc07a-preview-brodyd-1754925111-e44a21e.mintlify.app/oss/python/overview

This PR does two main things:

1. It reworks the `builder.py` file to generate JS and Python versions
of the docs.
2. It adds CSS and JS files for conditionally hiding the language
picker.

The requirements for the picker are:
- Show the picker when viewing the LangGraph docs
- Hide the picker when viewing the LangGraph Platform or LangChain Labs
docs

## Review requests + info

We're doing custom things on top of Mintlify's stuff. It would be great
if someone can help verify the changes to the `builder.py` file are
acceptable, and although it's not the most elegant solution, that the
CSS and JS files are an acceptable though likely temporary solution.

I tried to update and fix tests appropriately but I really need someone
to validate those changes are good. I haven't fixed/updated unit tests
like this before.

## Todos

- [x] Get this PR reviewed and implement any feedback
- [x] Fix tests
- [ ] Merge with #109
- [ ] 🎉

---------

Co-authored-by: Lauren Hirata Singh <[email protected]>
Copy link

Preview ID generated: preview-brodyd-1755288010-7f05e2e

Copy link

Preview ID generated: preview-brodyd-1755291374-9557e7d

Copy link

Preview ID generated: preview-brodyd-1755291711-8aaeb66

Copy link

Preview ID generated: preview-brodyd-1755548353-a5af602

Copy link

Preview ID generated: preview-brodyd-1755551180-a441ccb

Copy link

Preview ID generated: preview-brodyd-1755621941-8e6ec8d

Copy link

Preview ID generated: preview-brodyd-1755635447-9d1b028

Copy link

Preview ID generated: preview-brodyd-1755706430-59e5efd

@beklapko beklapko marked this pull request as ready for review August 20, 2025 16:31
Copy link

Preview ID generated: preview-brodyd-1755879478-9f01bc4

Copy link

Preview ID generated: preview-brodyd-1755888944-53bdddd

@beklapko
Copy link
Contributor Author

Got the 👍 to merge in Slack so 🚀

@beklapko beklapko merged commit 53f1cf9 into main Aug 22, 2025
14 of 15 checks passed
@beklapko beklapko deleted the brody/doc-60-port-integrations-docs branch August 22, 2025 18:58
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