Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feature: add docs for extra fields #11814

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

Merged
merged 1 commit into from
Jul 2, 2019

Conversation

Simperfit
Copy link
Contributor

Fix #11810

@Simperfit Simperfit force-pushed the add-docs-for-extra-fields branch from 73e4f2f to 879e1fe Compare June 23, 2019 14:16
@xabbuh xabbuh added this to the 4.4 milestone Jun 24, 2019

**type**: ``array`` **default**: ``null``

This is the custom fields to pull from the LDAP server. If the fields does not
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Defines the custom fields...

... If any field does not exist, an Invalid... will be thrown.

@javiereguiluz javiereguiluz changed the base branch from master to 4.4 July 2, 2019 06:53
@javiereguiluz javiereguiluz force-pushed the add-docs-for-extra-fields branch from 879e1fe to 76dadf8 Compare July 2, 2019 06:53
@javiereguiluz javiereguiluz merged commit 76dadf8 into symfony:4.4 Jul 2, 2019
javiereguiluz added a commit that referenced this pull request Jul 2, 2019
This PR was submitted for the master branch but it was merged into the 4.4 branch instead (closes #11814).

Discussion
----------

feature: add docs for extra fields

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Fix #11810

Commits
-------

76dadf8 feature: add docs for extra fields
@javiereguiluz
Copy link
Member

Thanks Hamza for this contribution! I made Ryan's suggested changes while merging and I also added the missing "versionadded:: 4.4" directive.

@Simperfit Simperfit deleted the add-docs-for-extra-fields branch July 2, 2019 06:56
@Simperfit
Copy link
Contributor Author

Thanks Javier, I forgot about Ryan's suggestion.

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.

6 participants