Skip to content

Conversation

@abdelrahmanAbouelkheir
Copy link
Contributor

@abdelrahmanAbouelkheir abdelrahmanAbouelkheir commented Mar 17, 2024

Description

Fixed grammar mistake by removing an extra 'a'.

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

Umbraco CMS v13

@abdelrahmanAbouelkheir abdelrahmanAbouelkheir changed the title Fix grammar mistake fix grammar Mar 17, 2024
@abdelrahmanAbouelkheir abdelrahmanAbouelkheir changed the title fix grammar fix grammar mistake Mar 17, 2024
Copy link
Contributor

@alina-tincas alina-tincas left a comment

Choose a reason for hiding this comment

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

Hi @abdelrahmanAbouelkheir thank you for the PR 🙌

I have noticed as well that if you head to https://github.com/umbraco/UmbracoDocs/pull/5960/files you can see a warning mentioning to write shorter sentences, less than 25 words. This is one of our styleguides to make our documentation more readable: https://docs.umbraco.com/welcome/contribute/style-guide#long-sentences

image

I have made a suggestion based on this, so I was wondering if you can take a look at my suggestions and let me know what you think? 😊

Also, I was wondering if you would be up for making the same changes for v12 and v10 of CMS as well, just to keep the content consistent throughout our documentation versions? 💡

* To return alternative templates depending on some custom business logic

This replacement of the default controller can either be made 'globally' for all requests (see last example) or by _'hijacking'_ requests for types of pages based on their specific Document Type following a this controller naming convention: `[DocumentTypeAlias]Controller`.
This replacement of the default controller can either be made 'globally' for all requests (see last example) or by _'hijacking'_ requests for types of pages based on their specific Document Type following this controller naming convention: `[DocumentTypeAlias]Controller`.
Copy link
Contributor

Choose a reason for hiding this comment

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

resolving a styleguide to have shorter sentences less than 25 words as well: https://docs.umbraco.com/welcome/contribute/style-guide#long-sentences
image

Suggested change
This replacement of the default controller can either be made 'globally' for all requests (see last example) or by _'hijacking'_ requests for types of pages based on their specific Document Type following this controller naming convention: `[DocumentTypeAlias]Controller`.
This replacement of the default controller can be made 'globally' for all requests (see last example). It can also be by _'hijacking'_ requests for types of pages based on their specific Document Type following this controller naming convention: `[DocumentTypeAlias]Controller`.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @alina-tincas thanks for the suggestion
I will fix them in version 10 and 12 right away

Copy link
Contributor

Choose a reason for hiding this comment

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

thank you for making the changes #h5YR

As you have created a new PR with new changes for the other versions in #5964 am closing this one 💪

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