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]: Validation message not working in slideout #15285

Closed
WHITE-developer opened this issue Jul 2, 2024 · 6 comments
Closed

[5.x]: Validation message not working in slideout #15285

WHITE-developer opened this issue Jul 2, 2024 · 6 comments
Assignees
Labels

Comments

@WHITE-developer
Copy link

What happened?

Description

Continuing on #15161, I think the validation is still not working when you are (multiple levels deep) in slideouts.

Steps to reproduce

  1. Open an entry in slideout
  2. In that entry we have the matrix block like in [5.x]: Matrix in card view doesn't show validation errors from card #15161, with the required field that is empty
  3. Try to save that entry --> No validation, just a message "Couldn't save entry"

Expected behavior

You should see the validations

Actual behavior

Message "Couldn't save entry"
image
image

Craft CMS version

5.2.4.1

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

No response

@brandonkelly
Copy link
Member

This should be fixed for Craft 5.3 via #15165. Would you mind testing to verify?

To test, change your craftcms/cms requirement in composer.json to:

"craftcms/cms": "5.3.x-dev as 5.3.0-alpha",

Then run composer update.

@WHITE-developer
Copy link
Author

Yes, except that my blocks don't have, I guess, a title?
And opening a block doesn't show which field is wrong.
image

@brandonkelly
Copy link
Member

We’re not able to reproduce that. Can you please send a database backup and your Composer files into [email protected], and point out where that issue is happening?

@WHITE-developer
Copy link
Author

You've got mail 📫

@brandonkelly
Copy link
Member

@WHITE-developer Thanks, we were able to reproduce with that. We made some adjustments (#15298) and it should be working as expected now. Run composer update if you want to test.

@WHITE-developer
Copy link
Author

Yes, perfect 👌. Thanks!

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