Skip to content

Conversation

@erikjanwestendorp
Copy link
Contributor

Description

Updates:

  • write shorter sentence

Type of suggestion

  • Styleguide

Product & version (if relevant)

Umbraco v10, 11, 13 & 13

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 @erikjanwestendorp thank you for the PR as always #h5YR

I have a suggestion, can you please take a look at it and let me know what you think? If you agree with it, can you please change it for all versions? 😊

erikjanwestendorp and others added 2 commits December 4, 2023 12:23
Co-authored-by: Alina-Magdalena Tincas <83591955+alina-tincas@users.noreply.github.com>
@erikjanwestendorp
Copy link
Contributor Author

@alina-tincas Thanks for your review, just updated the PR with your suggestion. Also updated version 11, 12 & 13.

@alina-tincas
Copy link
Contributor

Awesome, thank you 🙌

All looks good now so am merging it in 💪

@alina-tincas alina-tincas merged commit a2769b2 into umbraco:main Dec 4, 2023
@erikjanwestendorp erikjanwestendorp deleted the update-data-type branch December 4, 2023 12:36
@alina-tincas
Copy link
Contributor

alina-tincas commented Dec 5, 2023

@erikjanwestendorp v11 is currently EOL . So in this case it is not necessary anymore to make any changes to v11 docs going forward 😊

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