Skip to content

Conversation

@homestar9
Copy link
Contributor

See my comment on #45

@lmajano lmajano merged commit 109095b into coldbox-modules:development Jun 13, 2023
@lmajano
Copy link
Contributor

lmajano commented Jun 13, 2023

Would be great to have a note about this in the docs. Can you assist?

@homestar9
Copy link
Contributor Author

Sure thing, I will be happy to help. At first, I thought it might be a good idea to include a note at the top of the https://coldbox-validation.ortusbooks.com/overview/valid-constraints page, but some constraints like required and udf don't ignore nulls or empty strings. Therefore, I believe we should add a note for each constraint that does ignore null/empty next to their description. It will take longer, but I think it will be more beneficial for end-users. I will add this item to my "todo" list.

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

Successfully merging this pull request may close these issues.

2 participants