-
Notifications
You must be signed in to change notification settings - Fork 302
feat: Update sales agent deployment doc #2060
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR restructures the Sales Agent deployment documentation by removing the old deployment.md file and creating a new best-practices section with comprehensive deployment guides for multiple platforms (Ubuntu Server with PM2, AWS Amplify, and Cloudflare Pages). It also adds production build instructions to the installation documentation.
- Removes standalone deployment.md and migrates content to a better-organized best-practices structure
- Adds platform-specific deployment guides for Ubuntu/PM2, AWS Amplify, and Cloudflare Pages
- Updates installation.md with production build instructions
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| products/sales-agent/installation.md | Adds "Build code for Production" section with pnpm build command |
| products/sales-agent/deployment.md | Removes old deployment documentation (content migrated to best-practices) |
| products/sales-agent/best-practices/index.md | Creates new best-practices section index |
| products/sales-agent/best-practices/app-deployment/index.md | Creates app deployment subsection with overview and references |
| products/sales-agent/best-practices/app-deployment/hosted-with-ubuntu-server.md | Adds Ubuntu Server deployment guide using PM2 process manager |
| products/sales-agent/best-practices/app-deployment/cloudflare.md | Adds Cloudflare Pages deployment guide with local and GitHub Actions workflows |
| products/sales-agent/best-practices/app-deployment/aws.md | Adds AWS Amplify deployment guide |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
products/sales-agent/best-practices/app-deployment/hosted-with-ubuntu-server.md
Outdated
Show resolved
Hide resolved
products/sales-agent/best-practices/app-deployment/hosted-with-ubuntu-server.md
Outdated
Show resolved
Hide resolved
products/sales-agent/best-practices/app-deployment/cloudflare.md
Outdated
Show resolved
Hide resolved
products/sales-agent/best-practices/app-deployment/cloudflare.md
Outdated
Show resolved
Hide resolved
products/sales-agent/best-practices/app-deployment/cloudflare.md
Outdated
Show resolved
Hide resolved
products/sales-agent/best-practices/app-deployment/cloudflare.md
Outdated
Show resolved
Hide resolved
8eced49 to
a2aa672
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
products/sales-agent/best-practices/app-deployment/hosted-with-ubuntu-server.md
Outdated
Show resolved
Hide resolved
3d7c76f to
7551910
Compare
Isengo1989
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx for the PR @rittou 👍
No description provided.