Re: [GitHub] cloudstack-docs-admin pull request: CLOUDSTACK-6634

2014-08-20 Thread Rajani Karuturi
Thanks pdion. ~Rajani On Wed, Aug 20, 2014 at 5:20 PM, Pierre-Luc Dion wrote: > Thanks Seb, I've forgot to do this. > > Rajani, I've just fixed the 'new line'. all version are updated with this > new LDAP doc, there is a variable delay between git push and readthedocs > build get completed.

Re: [GitHub] cloudstack-docs-admin pull request: CLOUDSTACK-6634

2014-08-20 Thread Pierre-Luc Dion
Thanks Seb, I've forgot to do this. Rajani, I've just fixed the 'new line'. all version are updated with this new LDAP doc, there is a variable delay between git push and readthedocs build get completed. Thanks, *Pierre-Luc DION* Architecte de Solution Cloud | Cloud Solutions Architect t 855

Re: [GitHub] cloudstack-docs-admin pull request: CLOUDSTACK-6634

2014-08-20 Thread Sebastien Goasguen
Pdion891, if you amend the commit message with something like: this closes #15 the pr should get closed automatically. On Aug 19, 2014, at 8:39 PM, pdion891 wrote: > Github user pdion891 commented on the pull request: > > > https://github.com/apache/cloudstack-docs-admin/pull/15#issuecomme

Re: [GitHub] cloudstack-docs-admin pull request: CLOUDSTACK-6634

2014-08-19 Thread Rajani Karuturi
Hi pdion, Thanks for taking care. I closed the pull request. Can you check the formatting for the default value of ldap.username.attribute? I think it has gone to a new line. Also, I see the new doc only for 4.4[1]. where as master[2] and 4.3[3] still has the old one. Why is it so? [1] http://doc