Currently, the config.py file doesn’t have detailed information about the 
parameters. Do we need to update that file to include the detailed information?

Regards,
Abhilasha


> On 27-Apr-2020, at 6:39 PM, Akshay Joshi <akshay.jo...@enterprisedb.com> 
> wrote:
> 
> Hi Dave
> 
> On Mon, Apr 27, 2020 at 6:23 PM Dave Page <dave.p...@enterprisedb.com 
> <mailto:dave.p...@enterprisedb.com>> wrote:
> Hi Akshay,
> 
> I'm not sure that should even be there. We document other config options in 
> the default file and import that into the docs don't we?
> 
>    Yes, we have all the config options in the default file(config.py). But 
> the end-user need some reference in the documentation about LDAP, we can do 
> one thing to provide the link of the config.py as a reference for the 
> parameters. Please check the below screenshot what I am suggesting: 
>         <Screenshot 2020-04-27 at 6.31.57 PM.png>
> 
> 
> On Mon, Apr 27, 2020 at 1:51 PM Akshay Joshi <akshay.jo...@enterprisedb.com 
> <mailto:akshay.jo...@enterprisedb.com>> wrote:
> Hi Dave/Abhilasha
> 
> I have checked the documentation, can we have LDAP parameter in tabular 
> format for more readability. Below is the screenshot of the current view in 
> browser:
> 
> <LDAP_Parameters.png>
>  
> 
> On Mon, Apr 27, 2020 at 5:59 PM Abhilasha Narendra 
> <abhilasha.naren...@enterprisedb.com 
> <mailto:abhilasha.naren...@enterprisedb.com>> wrote:
> PFA the updated patch.
> 
> Regards,
> Abhilasha
> 
> 
> 
> 
> 
>> On 27-Apr-2020, at 5:41 PM, Abhilasha Narendra 
>> <abhilasha.naren...@enterprisedb.com 
>> <mailto:abhilasha.naren...@enterprisedb.com>> wrote:
>> 
>> Apologies for missing the file.
>> 
>> PFA the updated patch.
>> 
>> Regards,
>> Abhilasha
>> 
>> 
>> 
>> <rm5348.patch>
>> 
>> 
>> 
>> 
>>> On 27-Apr-2020, at 5:38 PM, Akshay Joshi <akshay.jo...@enterprisedb.com 
>>> <mailto:akshay.jo...@enterprisedb.com>> wrote:
>>> 
>>> Hi Abhilasha
>>> 
>>> Getting below error:
>>> 
>>> Warning, treated as error: 
>>> /Users/akshayjoshi/Developement/pgAdmin4/docs/en_US/getting_started.rst:31:toctree
>>>  contains reference to nonexisting document ‘enabling_ldap_authentication’
>>> 
>>> I think you forgot to add ‘enabling_ldap_authentication’ file in the patch.
>>> 
>>> 
>>> On Mon, Apr 27, 2020 at 3:36 PM Abhilasha Narendra 
>>> <abhilasha.naren...@enterprisedb.com 
>>> <mailto:abhilasha.naren...@enterprisedb.com>> wrote:
>>> 
>>> Hi Hackers,
>>> 
>>> I have updated the documentation for LDAP authentication support in pgAdmin.
>>> 
>>> PFA the patch.
>>> 
>>> Regards,
>>> Abhilasha
>>> 
>>> 
>>> -- 
>>> Thanks & Regards
>>> Akshay Joshi
>>> Sr. Software Architect
>>> EnterpriseDB Software India Private Limited
>>> Mobile: +91 976-788-8246
>> 
> 
> 
> 
> -- 
> Thanks & Regards
> Akshay Joshi
> Sr. Software Architect
> EnterpriseDB Software India Private Limited
> Mobile: +91 976-788-8246
> 
> 
> -- 
> Dave Page
> VP & Chief Architect, Database Infrastructure
> EnterpriseDB: http://www.enterprisedb.com <http://www.enterprisedb.com/>
> The Enterprise PostgreSQL Company
> 
> Blog: http://pgsnake.blogspot.com <http://pgsnake.blogspot.com/>
> Twitter: @pgsnake
> 
> 
> -- 
> Thanks & Regards
> Akshay Joshi
> Sr. Software Architect
> EnterpriseDB Software India Private Limited
> Mobile: +91 976-788-8246

Reply via email to