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

Skip to content

[Ldap] [Doc] Update README.md #18408

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
Apr 3, 2016
Merged

[Ldap] [Doc] Update README.md #18408

merged 1 commit into from
Apr 3, 2016

Conversation

csarrazi
Copy link
Contributor

@csarrazi csarrazi commented Apr 1, 2016

Q A
Branch? master
Bug fix? yes (doc fix)
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #18387
License MIT
Doc PR

Fixes a typo from #18387

@csarrazi csarrazi changed the title Update README.md [Ldap] [Doc] Update README.md Apr 1, 2016
@xabbuh
Copy link
Member

xabbuh commented Apr 1, 2016

I think we should better remove the link for the moment as the documentation isn't available yet.

@csarrazi
Copy link
Contributor Author

csarrazi commented Apr 1, 2016

Well, the doc PR is in progress, as mentioned here

@javiereguiluz
Copy link
Member

@xabbuh the first version of the doc will be available very soon, so I think we could merge this fix.

@fabpot
Copy link
Member

fabpot commented Apr 3, 2016

Thank you @csarrazi.

@fabpot fabpot merged commit 579e8d4 into symfony:master Apr 3, 2016
fabpot added a commit that referenced this pull request Apr 3, 2016
This PR was merged into the 3.1-dev branch.

Discussion
----------

[Ldap] [Doc] Update README.md

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes (doc fix)
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #18387
| License       | MIT
| Doc PR        |

Fixes a typo from #18387

Commits
-------

579e8d4 Update README.md
@csarrazi csarrazi deleted the patch-1 branch May 3, 2016 16:31
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.

5 participants