Re: [pgAdmin][RM5348]LDAP Authentication Support - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgAdmin][RM5348]LDAP Authentication Support
Date
Msg-id CA+OCxow2FVwDG+nSZPCL_Vx1mY=-ej1DFMaSQq3+Y2Hi1-EhiQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][RM5348]LDAP Authentication Support  (Abhilasha Narendra <abhilasha.narendra@enterprisedb.com>)
Responses Re: [pgAdmin][RM5348]LDAP Authentication Support  (Abhilasha Narendra <abhilasha.narendra@enterprisedb.com>)
List pgadmin-hackers
My point is that we should be consistent. If we add detailed docs for those config parameters, let's add them for everything (not necessarily today or this release of course).

I do agree with Akshay that this may work better as a table.

It shouldn't hold up the release though.

On Mon, Apr 27, 2020 at 2:18 PM Abhilasha Narendra <abhilasha.narendra@enterprisedb.com> wrote:
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.joshi@enterprisedb.com> wrote:

Hi Dave

On Mon, Apr 27, 2020 at 6:23 PM Dave Page <dave.page@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.joshi@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.narendra@enterprisedb.com> wrote:
PFA the updated patch.

Regards,
Abhilasha





On 27-Apr-2020, at 5:41 PM, Abhilasha Narendra <abhilasha.narendra@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.joshi@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.narendra@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
The Enterprise PostgreSQL Company

Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake


--
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
The Enterprise PostgreSQL Company

Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin][RM4504] Make like options disable if Relation is not selected.
Next
From: Aditya Toshniwal
Date:
Subject: [pgAdmin][RM5441] Not able to find path toast tables under pg_toast schema