Skip to content
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.

Sign up for GitHub

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

Jump to bottom

broken LangGraph docs link #26438

Merged
merged 1 commit into from
Sep 14, 2024
Merged

broken LangGraph docs link #26438

merged 1 commit into from
Sep 14, 2024

Conversation

jessicaou
Copy link
Contributor

@jessicaou jessicaou commented Sep 13, 2024 •
edited by eyurtsev
Loading

Update broken langgraph link in the README.md file

@vercel Vercel
Copy link

vercel bot commented Sep 13, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 13, 2024 6:08pm

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Sep 13, 2024
eyurtsev
eyurtsev requested changes Sep 13, 2024
View reviewed changes
Copy link
Collaborator

@eyurtsev eyurtsev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link intentionally links to the concepts. It doesn't look broken

@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Sep 13, 2024
@eyurtsev
Copy link
Collaborator

Ahhh this is the README

@hwchase17 hwchase17 merged commit 9c6aa3f into langchain-ai:master Sep 14, 2024
11 checks passed
Sheepsta300 pushed a commit to Sheepsta300/langchain that referenced this pull request Oct 1, 2024
@jessicaou @Sheepsta300
broken LangGraph docs link (langchain-ai#26438)
3f18e38 
Update broken langgraph link in the README.md file

Co-authored-by: Jess Ou <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eyurtsev eyurtsev eyurtsev approved these changes

Assignees
No one assigned
Labels
🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder lgtm PR looks good. Use to confirm that a PR is ready for merging. size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
PR Reviews
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jessicaou @eyurtsev @hwchase17