Skip to content
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

[5.x]: Matrix in card view doesn't show validation errors from card #15161

Closed
WHITE-developer opened this issue Jun 7, 2024 · 3 comments
Closed
Assignees
Labels

Comments

@WHITE-developer
Copy link

What happened?

Description

When saving an entry with a matrix in card view and one of the cards has a validation error, there is no indication which card has the error
image
image

Steps to reproduce

  1. Matrix in Card view
  2. Have it saved correctly
  3. Make a field required in the matrix
  4. Try resaving the entry

Expected behavior

Some indication where the error is

Actual behavior

No indication except on which tab the error is

Craft CMS version

5.1.9 & 5.2.0-beta.3

PHP version

8.3.7

Operating system and version

No response

Database type and version

MySQL 8.0.37

Image driver and version

No response

Installed plugins and versions

@i-just i-just self-assigned this Jun 7, 2024
@brandonkelly brandonkelly assigned brandonkelly and unassigned i-just Jun 7, 2024
brandonkelly added a commit that referenced this issue Jun 7, 2024
@brandonkelly
Copy link
Member

Thanks for pointing that out! This is now resolved for Craft 5.2, which will start displaying entry cards in an error state when they contain validation errors:

A Matrix field that contains a nested entry that identifies it has validation errors

@WHITE-developer
Copy link
Author

This seems not to work when in a x levels deep slideout, just getting a "Couldn’t save entry." message there, but I guess it's in progress for 5.3?

@brandonkelly
Copy link
Member

@WHITE-developer Can you post a new issue with full steps to reproduce + expected/actual results?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants