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

📝(content) Content Change from site/en/enterprise/app-distribution.md #632

Closed
wants to merge 3 commits into from
Closed

📝(content) Content Change from site/en/enterprise/app-distribution.md #632

wants to merge 3 commits into from

Conversation

Light13008
Copy link

@Light13008 Light13008 commented Jul 29, 2023 •
edited
Loading

Please describe what your Pull Request does
It fixes the typo correction from site/en/enterprise/app-distribution.md

A clear and concise description of what your Pull Request does.

The file present in the site/en/enterprise/app-distribution.md has a typo mistake the pull request does the typo correction from webapps to web apps.

Please tag each issue this Pull Request resolves
Add transform for webapp->web app #416

Resolves #416

@Light13008 Light13008 requested a review from a team as a code owner July 29, 2023 06:28
@Light13008 Light13008 changed the title (typo was corrected)webapps was changed to web app 📝 - Documentation/content change Jul 29, 2023
@Light13008 Light13008 changed the title 📝 - Documentation/content change 📝 - Documentation/content change site/en/enterprise/app-distribution.md Jul 29, 2023
@Light13008 Light13008 changed the title 📝 - Documentation/content change site/en/enterprise/app-distribution.md 📝 - content change site/en/enterprise/app-distribution.md Jul 29, 2023
@Light13008 Light13008 changed the title 📝 - content change site/en/enterprise/app-distribution.md 📝 - content change (site/en/enterprise/app-distribution.md) Jul 29, 2023
@Light13008 Light13008 changed the title 📝 - content change (site/en/enterprise/app-distribution.md) 📝 content change (site/en/enterprise/app-distribution.md) Jul 29, 2023
@Light13008 Light13008 changed the title 📝 content change (site/en/enterprise/app-distribution.md) 📝 (content) Content Change (site/en/enterprise/app-distribution.md) Jul 29, 2023
@Light13008 Light13008 changed the title 📝 (content) Content Change (site/en/enterprise/app-distribution.md) 📝 (content) Content Change from site/en/enterprise/app-distribution.md Jul 29, 2023
@Light13008 Light13008 changed the title 📝 (content) Content Change from site/en/enterprise/app-distribution.md 📝(content)Content Changesite/en/enterprise/app-distribution.md Jul 29, 2023
@Light13008 Light13008 changed the title 📝(content)Content Changesite/en/enterprise/app-distribution.md 📝(content)contentchangesite/en/enterprise/app-distribution.md Jul 29, 2023
@Light13008 Light13008 changed the title 📝(content)contentchangesite/en/enterprise/app-distribution.md 📝(content) Content Change from site/en/enterprise/app-distribution.md Jul 29, 2023
@Snugug
Copy link
Collaborator

Snugug commented Oct 2, 2023

Thanks for taking the time to file this PR. For small often miss-spelled words, or words with multiple accepted spellings but we only want to make one "correct", we prefer transforms (like the one for ChromeOS) instead of fixing individual instances in Markdown files. When we decide on an official spelling, we'll add a transform for webapp/web app.

Thanks again for taking the time!

@Snugug Snugug closed this Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add transform for webapp->web app
2 participants
@Light13008 @Snugug