Skip to content

Conversation

@eshanrnh
Copy link
Contributor

@eshanrnh eshanrnh commented Aug 19, 2024

Description

Added how to configure and use conditions on Workflows in Umbraco Forms

Type of suggestion

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

Product & version (if relevant)

Forms v10,12,13,14

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.

This is written as a guide.
Shouldn't it be added to a list instead? 🤔

Co-authored-by: sofietoft <stk@umbraco.com>
@eshanrnh
Copy link
Contributor Author

@sofietoft, I wrote it as a guide to match the rest of the article. Changing this part to a step-by-step or list format means we'll have to change the entire article to keep it consistent 🙈 The other articles in the Editor section are also written as guides, so if we decide to change this article to a step-by-step or list format, we might have to consider those as well.

@sofietoft
Copy link
Contributor

@sofietoft, I wrote it as a guide to match the rest of the article. Changing this part to a step-by-step or list format means we'll have to change the entire article to keep it consistent 🙈 The other articles in the Editor section are also written as guides, so if we decide to change this article to a step-by-step or list format, we might have to consider those as well.

Alright, that makes sense 😁

I do think thought that we at some point should ensure that whenever we have guides with steps, that the steps should be put into ordered lists. It's much easier to follow 💪🏻

@eshanrnh
Copy link
Contributor Author

Yup, I totally agree 💪 Merging this PR.

@eshanrnh eshanrnh merged commit 440d174 into main Aug 22, 2024
@eshanrnh eshanrnh deleted the conditional-workflow branch August 22, 2024 08:23
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