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

Correct ldap authenticator type in example config. #98

Merged
merged 1 commit into from Apr 8, 2015

Conversation

@dctrud
Copy link
Contributor

dctrud commented Apr 8, 2015

Thanks to @nsoranzo for spotting.

Thanks to @nsoranzo for spotting.
@martenson

This comment has been minimized.

Copy link
Member

martenson commented Apr 8, 2015

👍

martenson added a commit that referenced this pull request Apr 8, 2015
Correct ldap authenticator type in example config.
@martenson martenson merged commit f49abbd into galaxyproject:dev Apr 8, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@dannon

This comment has been minimized.

Copy link
Member

dannon commented Apr 8, 2015

@dctrud

This comment has been minimized.

Copy link
Contributor Author

dctrud commented Apr 9, 2015

Oh, sorry. I didn't see that one. Apologies!

@dannon

This comment has been minimized.

Copy link
Member

dannon commented Apr 9, 2015

No problem at all, I just saw the merge pop through on my phone and was really confused for a second since I was sure I'd already done this :)

@martenson

This comment has been minimized.

Copy link
Member

martenson commented Apr 9, 2015

lol, why did github show the diff then? I did vaguely remember I saw it before :D

@dannon

This comment has been minimized.

Copy link
Member

dannon commented Apr 9, 2015

Yeah, I know, that's why I was so confused too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.