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