Skip to content

docs(cloud): fix typos - #17247

Merged
shahednasser merged 1 commit into
medusajs:developfrom
prairiewriter:docs/fix-typos-cloud
Oct 23, 2026
Merged

shahednasser merged 1 commit into
medusajs:developfrom
prairiewriter:docs/fix-typos-cloud

Conversation

@prairiewriter

Copy link
Copy Markdown
Contributor

Summary

What — Fixes typos and one grammar issue across four Cloud docs pages:

  • comparison: "cacheing" → "caching"
  • environments/custom-domains: "envionment" → "environment" (two code comments)
  • environments/preview: "Everytime" → "Every time"
  • cli/commands/login: "A browser will open to login with your Cloud account" → "A browser will open so you can log in with your Cloud account" ("login" is the noun; "log in" is the verb)

Why — Typos in setup and deployment instructions undermine reader confidence, and the corrected code comments are likely to be copied into users' projects.

How — Content-only edits to MDX files. The two code-comment fixes don't affect any code. No code changes.

Testing — Reviewed the full diff. I didn't run yarn prep: the edited-dates step runs a Unix shell command that cmd.exe can't execute on Windows. Generated files such as llms-full.txt can be regenerated on merge.


Checklist

  • I have added a changeset for this PR — N/A, docs-only change
  • The changes are covered by relevant tests — N/A, content-only
  • I have verified the changes locally (diff reviewed)
  • I have linked the related issue(s) if applicable — N/A

@prairiewriter
prairiewriter requested a review from a team as a code owner October 11, 2026 15:07
@changeset-bot

changeset-bot Bot commented Oct 11, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 2b3f43d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@medusa-os-bot

medusa-os-bot Bot commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution! Initial automated review looks good.

The author corrects four typos across four Cloud documentation MDX pages: 'cacheing' → 'caching' (comparison page), 'envionment' → 'environment' (two shell code comments in custom-domains page), 'Everytime' → 'Every time' (preview page), and the noun 'login' corrected to the verb phrase 'log in' in the CLI login page. All corrections are accurate. The PR is docs-only with no code changes, no auto-generated files touched, no first-person prose, no cross-project links altered, and no new pages added. PR template is properly filled and the changeset/test N/A calls are appropriate for a content-only change.

Triggered by: new PR opened

@prairiewriter

Copy link
Copy Markdown
Contributor Author

For reference: I couldn't run yarn prep for this PR because it fails on Windows. I've filed the details in #17249. Generated files like llms-full.txt will need to be regenerated on merge.

@medusa-os-bot

medusa-os-bot Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution! Initial automated review looks good.

Verdict: Looks good for a final review by the team.

Corrects four typos across four Cloud documentation pages: "cacheing" → "caching", "envionment" → "environment" (two shell code comment occurrences), "Everytime" → "Every time", and the noun "login" to the verb phrase "log in".

Checked: prose accuracy, docs conventions, security, bugs.

Triggered by: manual workflow dispatch

@shahednasser shahednasser left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM thanks!

@shahednasser
shahednasser merged commit 7a22854 into medusajs:develop Oct 23, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants