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

Adding the payload parameter to the callback examples #6593

Merged
merged 1 commit into from
May 21, 2016

Conversation

jorgelbg
Copy link
Contributor

Updating the examples in http://symfony.com/doc/current/reference/constraints/Callback.html to show the existence of the $payload parameter, although at the moment none of the examples makes use of this parameter.

Q A
Doc fix? yes
New docs? no
Applies to 3.1,3.2
Fixed tickets #6185

@wouterj
Copy link
Member

wouterj commented May 21, 2016

👍

This is a great first step. Maybe in another PR, we can add a little description that this get the value of the payload option of the validation constraint.

@xabbuh
Copy link
Member

xabbuh commented May 21, 2016

👍

@wouterj
Copy link
Member

wouterj commented May 21, 2016

Good job, thanks!

@wouterj wouterj merged commit b1e586b into symfony:3.1 May 21, 2016
wouterj added a commit that referenced this pull request May 21, 2016
…jorgelbg)

This PR was merged into the 3.1 branch.

Discussion
----------

Adding the payload parameter to the callback examples

Updating the examples in http://symfony.com/doc/current/reference/constraints/Callback.html to show the existence of the `$payload` parameter, although at the moment none of the examples makes use of this parameter.

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 3.1,3.2
| Fixed tickets | #6185

Commits
-------

b1e586b Adding the payload parameter to the callback examples
wouterj added a commit that referenced this pull request May 21, 2016
@wouterj
Copy link
Member

wouterj commented May 21, 2016

Fyi, added a versionadded box indicating that this parameter is new in 3.1 in 8be3e3b

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