Skip to content

Conversation

@ImalshaD
Copy link
Contributor

@ImalshaD ImalshaD commented Oct 17, 2025

Related Issues

This pull request updates and expands the documentation for user flows in the identity server, focusing on clarity, usability, and new features. The changes include major rewrites for the flow builder guides, improved organization and navigation, and more practical, step-by-step instructions for building and customizing flows. The documentation now better explains flow inheritance, provides updated sample scenarios, and introduces a troubleshooting section.

Documentation improvements and expanded guides:

  • Major rewrite of build-a-flow.md to clarify the flow builder's purpose, add step-by-step instructions for creating, enabling, and inheriting flows, and provide detailed explanations of flow elements (steps, widgets, components). Flow inheritance for sub-organizations is now documented, including how to customize or revert flows for child orgs.
  • Updated index.md to provide a clearer overview of flows, improve descriptions of user journeys, and add a "Next steps" section for easier navigation to relevant guides.
  • Reworked invited-user-registration.md to use email OTP for identity verification (instead of SMS), clarify instructions for building the flow, and update screenshots and sample scenarios for accuracy and security.
  • Minor clarification in password-recovery.md for improved readability.

Navigation and structure:

  • Added "Troubleshooting" to the navigation in mkdocs.yml, making it easier for users to find help when encountering issues with flows.
  • Included a new troubleshooting guide in the documentation for user flows.

Copilot AI review requested due to automatic review settings October 17, 2025 02:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request comprehensively updates the flow documentation to improve clarity, usability, and coverage. The changes focus on rewriting core guides with step-by-step instructions, adding flow inheritance documentation for sub-organizations, and introducing a new troubleshooting section. The updates follow a more task-oriented approach with clearer action-first writing and better organization.

Key changes:

  • Major rewrite of flow builder guides with improved structure and step-by-step instructions
  • Introduction of troubleshooting guide for common Flow Builder errors
  • Updated sample scenarios across all flow types with clearer verification methods

Reviewed Changes

Copilot reviewed 8 out of 31 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
en/includes/guides/flows/trouble-shooting.md New troubleshooting guide documenting common Flow Builder validation errors and their solutions
en/includes/guides/flows/self-registration.md Rewritten to use Magic Link verification instead of SMS OTP, with simplified step-by-step instructions and expanded reference section
en/includes/guides/flows/password-recovery.md Updated sample use case to use SMS OTP, streamlined instructions, and added flow completion properties reference
en/includes/guides/flows/invited-user-registration.md Changed from SMS OTP to Email OTP verification with clearer configuration steps and new reference section
en/includes/guides/flows/index.md Improved overview with clearer descriptions and added "Next steps" section for better navigation
en/includes/guides/flows/build-a-flow.md Major rewrite with expanded explanations of flow elements, inheritance for sub-organizations, and improved structure
en/identity-server/next/mkdocs.yml Added troubleshooting guide to navigation structure
en/identity-server/next/docs/guides/flows/trouble-shooting.md Include file pointer for troubleshooting content

@ImalshaD ImalshaD requested a review from Copilot October 17, 2025 02:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 8 out of 31 changed files in this pull request and generated 16 comments.

- Self Registration: guides/flows/self-registration.md
- Password Recovery: guides/flows/password-recovery.md
- Invited User Registration: guides/flows/invited-user-registration.md
- Troubleshooting: guides/flows/trouble-shooting.md
Copy link
Contributor

@himeshsiriwardana himeshsiriwardana Oct 20, 2025

Choose a reason for hiding this comment

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

file name should be one word. troubleshooting.md

If you disable a customized flow, your users will automatically continue with the default experience until you’re ready to publish it again.

## Reference: Flow elements
## Flow inheritance for sub organizations
Copy link
Contributor

Choose a reason for hiding this comment

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

This needs to move from this doc to the organizations section. We have a separate section in Organizations to explain how inheritance work. This info is irrelevant to the flow doc.

Copy link
Contributor

Choose a reason for hiding this comment

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

When using a connection based sign up option (such as Google or Microsoft), configure the authorized redirect URL as: `https://accounts.asgardeo.io/t/{tenant-domain}/accounts/register`
When using a connection based sign up option (such as Google or Microsoft), configure the authorized redirect URL as: `https://{is_host}/t/{tenant-domain}/accounts/register`

## Reference
Copy link
Contributor

Choose a reason for hiding this comment

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

The flow completion properties shouldn't come as a reference. Let's make this a step (before enabling a flow step).

Step x : Configure flow completion properties

ImalshaD and others added 4 commits October 20, 2025 11:21
Improved some sections in the intro and added extra info on the flow behavior when enabled/disabled
minor improvements to the troubleshooting doc
@himeshsiriwardana himeshsiriwardana merged commit 315d0c2 into wso2:master Oct 20, 2025
1 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants