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

Add a CLI command that creates an empty LDAP config #11347

Merged
merged 1 commit into from
Sep 29, 2014
Merged

Add a CLI command that creates an empty LDAP config #11347

merged 1 commit into from
Sep 29, 2014

Conversation

mark0n
Copy link
Member

@mark0n mark0n commented Sep 29, 2014

This is needed to configure the user_ldap app without using the
wizard (e.g. using tools like Puppet).

@ghost
Copy link

ghost commented Sep 29, 2014

Thanks a lot for your contribution! Contributions to the core repo require a signed contributors agreement http://owncloud.org/contribute/agreement/ Alternatively you can add a comment here stating that this contribution is MIT licensed. Some more details about out pull request workflow can be found here: http://owncloud.org/code-reviews-on-github/

This is needed to configure the user_ldap app without using the
wizard (e.g. using tools like Puppet).
@scrutinizer-notifier
Copy link

The inspection completed: 4 new issues, 4 updated code elements

@MorrisJobke
Copy link
Contributor

cc @blizzz

@blizzz
Copy link
Contributor

blizzz commented Sep 29, 2014

Thank you for your contribution! :)

Tested, works and looks good 👍

@mark0n could you comment on #11347 (comment)?

@blizzz
Copy link
Contributor

blizzz commented Sep 29, 2014

@owncloud-bot this is ok to test

@ghost
Copy link

ghost commented Sep 29, 2014

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser/7610/

@mark0n
Copy link
Member Author

mark0n commented Sep 29, 2014

I emailed a signed copy of the contributor agreement to Frank last night.

@LukasReschke
Copy link
Member

@karlitschek Can you confirm? THX.

@karlitschek
Copy link
Contributor

confirmed :-)

@LukasReschke
Copy link
Member

👍

LukasReschke added a commit that referenced this pull request Sep 29, 2014
Add a CLI command that creates an empty LDAP config
@LukasReschke LukasReschke merged commit b13b865 into owncloud:master Sep 29, 2014
@mark0n mark0n deleted the create_empty_config branch September 30, 2014 01:20
@lock lock bot locked as resolved and limited conversation to collaborators Aug 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants