Skip to content

Conversation

@eshanrnh
Copy link
Contributor

@eshanrnh eshanrnh commented Sep 18, 2024

Description

Created a Tutorial on how to create a multi-page form using Umbraco Forms

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)

Anytime

@eshanrnh
Copy link
Contributor Author

The video cannot be added in the article since it uses Macro (discontinued from v14) to embed the Form in the RTE.

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.

Great tutorial 💪
I really like the "real life" example that you've chosen for the tutorial - makes it seem relevant!

I've added some suggestions here and there.
Also, a few additional thoughts:

  • Would be great with some more descriptive image captions, especially for the images of the "entire pages" after all fields are added. These images are a little confusing as they're not introduced in the text.
  • When introducing the condition - I think it would be great to explain the condition that's set. So, writing something like "Show this field if xx field is xx". Just to we do not rely only on the image/visuals for this.

Let me know if something doesn't make sense!

@eshanrnh
Copy link
Contributor Author

Thanks @sofietoft for the review and suggestions 🙌I've incorporated all of it. Let me know how it looks😀

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.

LGTM 💪
Super tutorial - and shame about the video material being outdated, but that's a task for another day 😅

@eshanrnh eshanrnh merged commit 4a2fa09 into main Sep 19, 2024
@eshanrnh eshanrnh deleted the multi-page-form branch September 19, 2024 12:17
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