docs: quick guide for public deployment#8712
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughA new documentation page was added, detailing how to deploy a Langflow server to the public internet using ngrok. The sidebar configuration was updated to include this new guide in the Deployment section, positioned after the Docker deployment documentation. Changes
Suggested labels
Suggested reviewers
✨ Finishing Touches🧪 Generate Unit Tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
|
Build successful! ✅ |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
docs/docs/Deployment/deployment-public-server.md (2)
46-48: Specify a language for this fenced block to satisfy markdown-lint (MD040).
markdownlint-cli2flags unlabeled code fences.
Because the block only illustrates console output, mark it explicitly astext(orbashif you prefer) to silence the warning and keep formatting consistent.-``` +```text Forwarding https://94b1-76-64-171-14.ngrok-free.app -> http://localhost:7860--- `18-38`: **Add a brief security warning about public exposure.** Opening a dev server to the internet without auth carries risk. A short Docusaurus *:::warning* admonition (or *:::tip* pointing to ngrok auth/static-domain docs) right after this numbered list would help readers avoid accidental data leaks. </blockquote></details> </blockquote></details> <details> <summary>📜 Review details</summary> **Configuration used: .coderabbit.yaml** **Review profile: CHILL** **Plan: Pro** <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 1903f607e8bb4ffa896ea9a5dec87e2fa66aa967 and 6ec864effd64286f900fd194af0456b5006b57c0. </details> <details> <summary>📒 Files selected for processing (2)</summary> * `docs/docs/Deployment/deployment-public-server.md` (1 hunks) * `docs/sidebars.js` (1 hunks) </details> <details> <summary>🧰 Additional context used</summary> <details> <summary>📓 Path-based instructions (2)</summary> <details> <summary>`docs/sidebars.js`: Sidebar configuration must be updated to reflect new or reorganized documentation sections, using the Docusaurus sidebar format.</summary> > `docs/sidebars.js`: Sidebar configuration must be updated to reflect new or reorganized documentation sections, using the Docusaurus sidebar format. - `docs/sidebars.js` </details> <details> <summary>`docs/**/*.{md,mdx}`: All documentation content must follow the markdown conventions outlined, including frontmatter with title, description, and sidebar_position, use of proper he...</summary> > `docs/**/*.{md,mdx}`: All documentation content must follow the markdown conventions outlined, including frontmatter with title, description, and sidebar_position, use of proper headers, admonitions (tip, warning, danger), and code block formatting. > Images should be stored in the 'static/img/' directory and referenced with descriptive alt text in markdown. - `docs/docs/Deployment/deployment-public-server.md` </details> </details><details> <summary>🪛 markdownlint-cli2 (0.17.2)</summary> <details> <summary>docs/docs/Deployment/deployment-public-server.md</summary> 46-46: Fenced code blocks should have a language specified null (MD040, fenced-code-language) </details> </details> </details> <details> <summary>⏰ Context from checks skipped due to timeout of 90000ms (1)</summary> * GitHub Check: build-and-deploy </details> <details> <summary>🔇 Additional comments (3)</summary><blockquote> <details> <summary>docs/docs/Deployment/deployment-public-server.md (2)</summary> `58-66`: **Endpoint path appears inconsistent – verify `/public_flow` vs `/playground`.** The paragraph says the Playground is exposed at `/public_flow/{flow-id}`, but the example URL shows `/playground/{flow-id}`. Confirm which path Langflow actually generates and adjust the text (and link text) for accuracy. --- `22-24`: **Double-check the command `uv run langflow run`.** If the intended command is `langflow run` (or `uv pip install` etc.), make sure this composite invocation is actually valid; otherwise users will hit a command-not-found error. </details> <details> <summary>docs/sidebars.js (1)</summary> `157-161`: **Sidebar entry looks correct.** ID, label, and position align with conventions and the new doc file. No issues spotted. </details> </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
aimurphy
left a comment
There was a problem hiding this comment.
This introduces some redundancies I think we should avoid.
|
Build successful! ✅ |
|
Build successful! ✅ |
Co-authored-by: April I. Murphy <36110273+aimurphy@users.noreply.github.com>
|
Build successful! ✅ |
|
Build successful! ✅ |
|
Build successful! ✅ |
aimurphy
left a comment
There was a problem hiding this comment.
A few final touchups, but LGTM!
Co-authored-by: April I. Murphy <36110273+aimurphy@users.noreply.github.com>
|
Build successful! ✅ |
|
Build successful! ✅ |
Make ngrok instructions their own page.
Add instructions for MCP server, API run endpoint, and Shareable Playground usage with public server.
Question: No ngrok auth added here
Summary by CodeRabbit