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

the expected error on the missing book title does not appear #1336

Open
CedCannes opened this issue Apr 3, 2021 · 0 comments
Open

the expected error on the missing book title does not appear #1336

CedCannes opened this issue Apr 3, 2021 · 0 comments

Comments

@CedCannes
Copy link
Contributor

Hi,
I followed the how to and I have an unexpected result
in docs/distribution/index.md
L629 Oops, we forgot to add the title. Submit the request anyway, you should get a 500 error with the following message:

This error does not appear because the title field is initialized to '' and not to null.

/** The title of this book. */
    public string $title = '';

In addition the documentation proposes to add a validator to prevent the error from happening again but doesn't add one to the title field mentioned above but expects the following exception to appear:

{
      "propertyPath": "title",
      "message": "This value should not be blank."
    }
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

No branches or pull requests

1 participant