Skip to content

[LiveComponent][Docs] Added guidance for rendering live form validation errors #2696

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

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

Precipitator80
Copy link

Q A
Bug fix? no
New feature? no
Docs? yes
Issues N/A
License MIT

This is a small documentation addition that helps developers show validation for any live form rows using the data-model attribute. As this is my first contribution, please excuse any initial errors and let me know how to make the addition clearer or fix any misunderstanding. Thank you!

@carsonbot carsonbot added docs Improvements or additions to documentation Status: Needs Review Needs to be reviewed labels Apr 16, 2025
@Kocal Kocal changed the title minor [Docs] Added guidance for rendering live form validation errors [LiveComponent][Docs] Added guidance for rendering live form validation errors Apr 16, 2025
Copy link
Member

@Kocal Kocal left a comment

Choose a reason for hiding this comment

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

Hi, congrats and thanks for your first contribution!

I'm not confident enough with LiveComponent, but to me it looks like tweaking an internal (but public) behavior to trigger a certain behavior, and I'm not sure if we really want to document that.

WDYT about it @kbond @smnandre ?

@Kocal Kocal added Status: Waiting Feedback Needs feedback from the author and removed Status: Needs Review Needs to be reviewed labels Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation LiveComponent Status: Waiting Feedback Needs feedback from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants