langchain-ai/docs
1
0
Fork 0
mirror of https://github.com/langchain-ai/docs.git synced 2026-07-22 12:25:55 -04:00
Code Issues 490 Packages Projects Releases Wiki Activity

[PR #125] [MERGED] Brody/docs versions #439

New Issue
Closed
opened 2026-02-17 17:20:21 -05:00 by yindo · 0 comments
yindo commented 2026-02-17 17:20:21 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/125
Author: @beklapko
Created: 8/6/2025
Status: ✅ Merged
Merged: 8/13/2025
Merged by: @beklapko

Base: migrate-lg ← Head: brody/docs-versions

📝 Commits (10+)

  • 037610f Initial pass on adding language picker/version
  • ee7d177 Remove backup file
  • 85b50fd Fix LangChain Labs dropdown
  • d1362c0 Merge branch 'migrate-lg' into brody/docs-versions
  • 8b9b7cc Change build and URL structure so only LangGraph has JS and Python versions
  • 9baefcd Merge branch 'brody/docs-versions' of https://github.com/langchain-ai/docs into brody/docs-versions
  • dd1586e Only show version selection for langgraph docs
  • ca5887e Put language docs at root, move oss folder and put docs in there to fix URL path
  • fa472f7 Simplify show/hiding logic
  • fac1b99 Bookmark changes to move version picker location

📊 Changes

6 files changed (+1365 additions, -426 deletions)

View changed files

📝 README.md (+8 -1)
📝 pipeline/core/builder.py (+301 -41)
📝 src/docs.json (+838 -332)
➕ src/hide-version-picker.css (+27 -0)
➕ src/hide-version-picker.js (+51 -0)
📝 tests/unit_tests/test_builder.py (+140 -52)

📄 Description

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

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/docs/pull/125 **Author:** [@beklapko](https://github.com/beklapko) **Created:** 8/6/2025 **Status:** ✅ Merged **Merged:** 8/13/2025 **Merged by:** [@beklapko](https://github.com/beklapko) **Base:** `migrate-lg` ← **Head:** `brody/docs-versions` --- ### 📝 Commits (10+) - [`037610f`](https://github.com/langchain-ai/docs/commit/037610f95bcc66c3ffa7f49e9fcce58f0f782678) Initial pass on adding language picker/version - [`ee7d177`](https://github.com/langchain-ai/docs/commit/ee7d177ee7714c8891bb1abb35a357dd37513407) Remove backup file - [`85b50fd`](https://github.com/langchain-ai/docs/commit/85b50fd4e6a0b1e2968a4bb80789e514c55ba67b) Fix LangChain Labs dropdown - [`d1362c0`](https://github.com/langchain-ai/docs/commit/d1362c06258613792870ec7dace65ce2ff92641d) Merge branch 'migrate-lg' into brody/docs-versions - [`8b9b7cc`](https://github.com/langchain-ai/docs/commit/8b9b7cc5d5d9ddd6434d9a7a97c948e1d2dcb670) Change build and URL structure so only LangGraph has JS and Python versions - [`9baefcd`](https://github.com/langchain-ai/docs/commit/9baefcd988c65080a347c3c7c3477cb9a30b5c86) Merge branch 'brody/docs-versions' of https://github.com/langchain-ai/docs into brody/docs-versions - [`dd1586e`](https://github.com/langchain-ai/docs/commit/dd1586e24c5cc407ad6d7537c7d70a5cc44d8e01) Only show version selection for langgraph docs - [`ca5887e`](https://github.com/langchain-ai/docs/commit/ca5887e9c1071daebf1f8256b011712cd46baa66) Put language docs at root, move oss folder and put docs in there to fix URL path - [`fa472f7`](https://github.com/langchain-ai/docs/commit/fa472f75a92fa3377d38460e70b7bc7e3d3e19cd) Simplify show/hiding logic - [`fac1b99`](https://github.com/langchain-ai/docs/commit/fac1b99fe9929574cb710653c948d2455fc2658e) Bookmark changes to move version picker location ### 📊 Changes **6 files changed** (+1365 additions, -426 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+8 -1) 📝 `pipeline/core/builder.py` (+301 -41) 📝 `src/docs.json` (+838 -332) ➕ `src/hide-version-picker.css` (+27 -0) ➕ `src/hide-version-picker.js` (+51 -0) 📝 `tests/unit_tests/test_builder.py` (+140 -52) </details> ### 📄 Description ## 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 https://github.com/langchain-ai/docs/pull/109 - [ ] 🎉 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-17 17:20:21 -05:00
yindo closed this issue 2026-02-17 17:20:21 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
deepagents
docs-infra
external
external
help wanted
integration
internal
internal
javascript
js/ts
langchain
langgraph
langsmith
needs-title
pull-request
Mirrored from GitHub Pull Request
 python
reference
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/docs#439
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?