Skip to content

Document the SecurityUserValueResolver & Psr7ServerRequestResolver #11467

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 23, 2019
Merged

Document the SecurityUserValueResolver & Psr7ServerRequestResolver #11467

merged 1 commit into from
Apr 23, 2019

Conversation

HypeMC
Copy link
Member

@HypeMC HypeMC commented Apr 23, 2019 •
edited
Loading

Fixes #10595.

@HypeMC HypeMC changed the base branch from master to 3.4 April 23, 2019 00:02
@HypeMC HypeMC mentioned this pull request Apr 23, 2019
Merged
@javiereguiluz javiereguiluz merged commit 0e1b386 into symfony:3.4 Apr 23, 2019
javiereguiluz added a commit that referenced this pull request Apr 23, 2019
@javiereguiluz
minor #11467 Document the SecurityUserValueResolver & Psr7ServerReque…
9713707 
…stResolver (HypeMC)

This PR was merged into the 3.4 branch.

Discussion
----------

Document the SecurityUserValueResolver & Psr7ServerRequestResolver

Fixes #10595.

Commits
-------

0e1b386 Document the SecurityUserValueResolver & Psr7ServerRequestResolver
@javiereguiluz javiereguiluz added this to the 3.4 milestone Apr 23, 2019
@javiereguiluz
Copy link
Member

@HypeMC thanks a lot for contributing this missing doc ... and congrats on your first Symfony Docs contribution!

javiereguiluz added a commit that referenced this pull request Apr 24, 2019
@javiereguiluz
minor #11468 Document the UserValueResolver (HypeMC)
708e98c 
This PR was merged into the 4.2 branch.

Discussion
----------

Document the UserValueResolver

Fixes #10595, followup to #11467.

**NOTE**: This PR has all the changes that #11467 has plus some symfony 4.2 specific additions, so #11467 should probably be merged first.

Commits
-------

b72a65e Document the UserValueResolver
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
HttpKernel Status: Reviewed
Projects
None yet
Milestone
3.4
Development

Successfully merging this pull request may close these issues.
3 participants
@HypeMC @javiereguiluz @carsonbot