Skip to content

Conversation

@alina-tincas
Copy link
Contributor

Description

What did you add/update/change?
Restructuring notification articles:

  • most articles had information already about them on API docs, so I removed those articles and added a link to API docs on the readme file. Made a CMS PR to add extra information that was on docs but not on API docs: Update v14 api docs with info from umbraco docs Umbraco-CMS#15977
  • added descriptions to the articles that were missing them
  • added articles that existed under using notifications but not mentioned in the readme file
  • moved some content from readme file to a new articles so that the readme file is showcasing an overview of all articles that "live" under it.

Type of suggestion

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

Product & version (if relevant)

cms v14, let me know if it makes sense to make these changes for the other versions as well?

Deadline (if relevant)

When should the content be published?
anytime

@alina-tincas alina-tincas marked this pull request as draft April 4, 2024 11:47
@alina-tincas alina-tincas marked this pull request as ready for review April 4, 2024 11:59
Copy link
Contributor

@jonat123 jonat123 left a comment

Choose a reason for hiding this comment

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

@alina-tincas LGTM 💪

@alina-tincas alina-tincas merged commit 4d238a5 into main Apr 9, 2024
@alina-tincas alina-tincas deleted the v14-restructure-notifications-articles branch April 10, 2024 14:00
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.

3 participants