Skip to content

Conversation

@eshanrnh
Copy link
Contributor

@eshanrnh eshanrnh commented Feb 10, 2025

Description

What did you add/update/change?

Type of suggestion

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

Product & version (if relevant)

Cloud

Deadline (if relevant)

Anytime

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

Looks good.
I've left a single comment - let me know what you think!

Comment on lines 47 to 48
1. When you want to add a user to your project from the Backoffice. This action involves sending an email to the new user. We have set up a fallback that ensures an email is sent even if you haven't configured the SMTP service yet. However, this fallback is **only** for inviting users to your project.
2. To set up the SMTP service for your Umbraco Cloud project if one of your Backoffice users forgets their password. The user will need to request a password reset, which will be sent via email. This will only work once you've configured the SMTP service.
Copy link
Contributor

Choose a reason for hiding this comment

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

This to me looks a little wierd in an ordered list 🤔
Not sure if an unordered list is better, or if the two scenarios could be set up in a better way 🤔

Thoughts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Makes sense. I have changed it to an unordered list and refined the text a bit more, so that there's not too much text to read 💪

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

Looks great 💪

@eshanrnh eshanrnh merged commit a065149 into main Feb 11, 2025
12 of 19 checks passed
@eshanrnh eshanrnh deleted the smtp-settings branch February 11, 2025 09:13
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