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

Add support for plain Symfony forms and validation #6

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ggioffreda
Copy link

By defaulting to the submitted form value this validator can be used with custom Symfony form types.

This is useful for example:

  • when it's not possible to post to the hardcoded "cf-turnstile-response" form field
  • when using a third party widget (like vue-turnstile), while still performing server side validation with Symfony validators
  • when the data is not submitted through a standard web request

By defaulting to the submitted form value this validator can be
used with custom Symfony form types.

This is useful for example:
- when it's not possible to post to the hardcoded
  "cf-turnstile-response" form field
- when using a third party widget (like vue-turnstile), while
  still performing server side validation with Symfony validators
- when the data is not submitted through a standard web request
@matthieu2607
Copy link
Collaborator

Hi @ggioffreda,

Thank you for your contribution.

I will consider your contribution quickly ;)

Copy link

sonarcloud bot commented Mar 23, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@fliespl
Copy link

fliespl commented Dec 6, 2024

I second this PR - currently it's hardcoded to field name: cf-turnstile-response, so example with "security" field name doesn't work.

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.

3 participants