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

Cloud CLI documentation #2086

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Cloud CLI documentation #2086

wants to merge 14 commits into from

Conversation

meganelacheny
Copy link
Collaborator

@meganelacheny meganelacheny commented Apr 17, 2024 •
edited

  • Create Cloud CLI section in Cloud documentation TOC
  • Document Cloud CLI in dedicated page
  • Update deployment guide in Cloud docs
  • Create deployment guide specifically for CLI
  • Add screenshots for CLI deployment guide

@meganelacheny meganelacheny added flag: merge pending release Pending Strapi Release internal PRs created by the Strapi core team pr: new content PRs for new product features or new documentation sections source: Strapi Cloud PRs/issues targeting the Strapi Cloud documentation labels Apr 17, 2024
@meganelacheny meganelacheny self-assigned this Apr 17, 2024
@vercel Vercel
Copy link

vercel bot commented Apr 17, 2024 •
edited

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

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2024 2:51pm

@meganelacheny meganelacheny marked this pull request as ready for review April 25, 2024 12:58
@meganelacheny meganelacheny requested review from pwizla and removed request for pwizla April 25, 2024 12:58
pwizla
pwizla reviewed Apr 25, 2024
View reviewed changes
docusaurus/docs/cloud/cli/cloud-cli.md Outdated
It is recommended to install Strapi locally only, which requires prefixing all of the following `strapi` commands with the package manager used for the project setup (e.g `npm run strapi help` or `yarn strapi help`) or a dedicated node package executor (e.g. `npx strapi help`).
:::

<!-- Do we need to repeat the prerequisites here?
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we don't (need to repeat them) 😊

pwizla
pwizla reviewed Apr 25, 2024
View reviewed changes
Copy link
Collaborator

@pwizla pwizla left a comment

Choose a reason for hiding this comment

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

Great job, Mégane, thanks! 🚀 Looks good to me

I only made some wording suggestions to avoid a possible confusion between code stored in a local git repository and code hosted in a remote repository (GitHub, GitLab…).

meganelacheny and others added 4 commits April 25, 2024 16:46
@meganelacheny @pwizla
Update wording for "hosted on remote git repo"
f7a5552 
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
@meganelacheny @pwizla
Improve wording again for "hosted on remote git repo"
4e11ec4 
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
@meganelacheny @pwizla
Improve wording again
dfc36ec 
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
@meganelacheny @pwizla
Update docusaurus/docs/cloud/cli/cloud-cli.md
29099e9 
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pwizla pwizla pwizla left review comments

Assignees

@meganelacheny meganelacheny

Labels
flag: merge pending release Pending Strapi Release internal PRs created by the Strapi core team pr: new content PRs for new product features or new documentation sections source: Strapi Cloud PRs/issues targeting the Strapi Cloud documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@meganelacheny @pwizla