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.

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

update broken links #445

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jannikmaierhoefer
Copy link

@jannikmaierhoefer jannikmaierhoefer commented Oct 31, 2024

Added absolute links since relative links are broken if the page is rendered on: https://js.reference.langfuse.com/


Important

Update broken relative links to absolute links in README.md.

  • Links:
    • Updated relative links to absolute links in README.md for langfuse, langfuse-node, langfuse-langchain, and langfuse-vercel packages.

This description was created by Ellipsis for 046baa0. It will automatically update as commits are pushed.

Added absolute links since relative links are broken if page is rendered on: https://js.reference.langfuse.com/
Copy link

vercel bot commented Oct 31, 2024

@jannikmaierhoefer is attempting to deploy a commit to the langfuse Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Disclaimer: Experimental PR review

PR Summary

Updates package links in README.md from relative paths to absolute GitHub URLs to fix broken links when documentation is rendered on js.reference.langfuse.com.

  • Changed package links in /README.md from relative paths (e.g. ./langfuse) to absolute GitHub URLs (e.g. https://github.com/langfuse/langfuse-js/tree/main/langfuse) for better cross-platform compatibility
  • Improves documentation accessibility when viewed on js.reference.langfuse.com

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

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.

1 participant