-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Escalation Events Properties and Form Styling Documentation #1047
Conversation
WalkthroughWalkthroughThe recent updates include improvements to the documentation, focusing on form layouts and workflow management. These changes introduce guidance on configuring side-by-side field layouts in forms using the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: .coderabbit.yaml
Files selected for processing (2)
- docs/Building_Diagrams/Forms.md (2 hunks)
- docs/Building_Diagrams/escalation_events.md (1 hunks)
Files skipped from review due to trivial changes (1)
- docs/Building_Diagrams/escalation_events.md
Additional comments: 2
docs/Building_Diagrams/Forms.md (2)
- 335-335: The heading "UI Schema Example" is correctly updated for consistency, aligning with the documentation's style guide.
- 351-398: The new section on configuring side-by-side field layout in forms is comprehensive, providing clear steps, an illustrative example, and an explanation of how the
ui:layout
attribute works. However, ensure that the example JSON is valid and correctly formatted to prevent confusion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be good after the side by side stuff is merged with the existing docs or removed.
Here's how to set up these components: | ||
|
||
**Define the Escalation ID**: Determine the task or process stage where an escalation might need to be triggered due to exceptional circumstances. and decide on a unique identifier for your escalation event. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe just a comma here: "exceptional circumstances. and decide on a unique identifier"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: .coderabbit.yaml
Files selected for processing (1)
- docs/Building_Diagrams/Forms.md (4 hunks)
Files skipped from review as they are similar to previous changes (1)
- docs/Building_Diagrams/Forms.md
Done, merged with existing one. |
Summary by CodeRabbit