Skip to content

Conversation

@kjac
Copy link
Contributor

@kjac kjac commented Jun 7, 2024

Description

The "Creating a user" article references obsolete and removed classes and service methods. It is also not at all about creating users - it is about adding users to groups 😆

I have rewritten the entire article to get the code samples etc. up to date with V14. I have also opted to rename the article to "Managing Users". Eventually we could add more samples to this - for example, something about creating users? 😛

Type of suggestion

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

Product & version (if relevant)

V14

Deadline (if relevant)

Whenever possible.

@jonat123
Copy link
Contributor

@kjac Thanks for creating this PR 😄

We will review it as soon as we can 😊

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.

@kjac Looks good!

Just have two suggestions 😄

SInce you are removing an article, we also need a redirect in place.

Can you add the following line to the .gitbook.yaml file:
reference/management/services/create-a-new-user: reference/management/services/managing-users.md

you can find the file in the 14 -> umbraco-cms folder 😄

kjac and others added 3 commits June 17, 2024 14:31
Co-authored-by: jonat123 <54025331+jonat123@users.noreply.github.com>
Co-authored-by: jonat123 <54025331+jonat123@users.noreply.github.com>
@kjac
Copy link
Contributor Author

kjac commented Jun 17, 2024

Done @jonat123 👍

# Conflicts:
#	14/umbraco-cms/.gitbook.yaml
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.

@kjac LGTM 💪

Merging it in!

@jonat123 jonat123 merged commit ee8b40c into umbraco:main Jun 18, 2024
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