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

feat(validation): validate query parameters with multiple values #5890

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

aegypius
Copy link
Contributor

@aegypius aegypius commented Oct 17, 2023

Ref #4798

Tries to fix query parameters with multiple values.

  • Fix tests
  • Add a test for differents multiple values allowed

@aegypius aegypius force-pushed the 4798-validate-parameters-with-multiple-values branch 2 times, most recently from 6fccee9 to db1c5e5 Compare October 27, 2023 12:46
@soyuka soyuka changed the title 4798 validate parameters with multiple values feat(validation): validate query parameters with multiple values Oct 28, 2023
Copy link

stale bot commented Dec 29, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 29, 2023
@soyuka soyuka added enhancement and removed stale labels Dec 29, 2023
@aegypius
Copy link
Contributor Author

aegypius commented Jan 4, 2024

this PR must be rebased on the new api-platform/parameter-validator component (see #6080)

@aegypius aegypius force-pushed the 4798-validate-parameters-with-multiple-values branch from ff6309f to 6d1e03e Compare January 4, 2024 21:29
@aegypius
Copy link
Contributor Author

aegypius commented Jan 5, 2024

this PR must be rebased on the new api-platform/parameter-validator component (see #6080)

Done !

@aegypius aegypius force-pushed the 4798-validate-parameters-with-multiple-values branch from 6d1e03e to c0a7490 Compare January 5, 2024 08:49
@aegypius aegypius force-pushed the 4798-validate-parameters-with-multiple-values branch from c0a7490 to 28425ca Compare January 6, 2024 10:00
@aegypius aegypius force-pushed the 4798-validate-parameters-with-multiple-values branch from 8ec5dcb to a8da9af Compare January 7, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants