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

[HttpKernel] Document supported types #[MapQueryParameter] #20783

Merged

Conversation

alamirault
Copy link
Contributor

@carsonbot carsonbot added this to the 6.4 milestone Mar 20, 2025
@alamirault alamirault changed the title Document supported types #[MapQueryParameter] Document supported types #[MapQueryParameter] Mar 20, 2025
@carsonbot carsonbot changed the title Document supported types #[MapQueryParameter] [HttpKernel] Document supported types #[MapQueryParameter] Mar 21, 2025
@javiereguiluz javiereguiluz merged commit 4dd7073 into symfony:6.4 Mar 21, 2025
3 checks passed
@javiereguiluz
Copy link
Member

Antoine, thanks for this!

While merging, I did some minor changes 😇 (0772b75) I remove the :class:... becaus ewe usually only include that the first time that class appears in the docs and there's an occurrence a few lines above the new docs. I also sorted the valid data types alphabetically.

@alamirault alamirault deleted the feature/20626-uid-query-parameter branch March 21, 2025 22:34
javiereguiluz added a commit that referenced this pull request Mar 24, 2025
…irault)

This PR was merged into the 7.3 branch.

Discussion
----------

[HttpKernel] Support Uid in `#[MapQueryParameter]`

When #20783 is merge we can document new supported type.

Fix #20626

Commits
-------

2a2faa4 [HttpKernel] Support Uid in #[MapQueryParameter]
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.

3 participants