Re: [pgAdmin][RM6231]- Add OS, Browser details in pgAdmin About us pop-up

Hi

On Fri, Jun 4, 2021 at 2:34 PM Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> Hello,
>
> I want to propose a suggestion here, If we can display configuration data
> in category/tab like below then it will make sense for the user (instead of
> dumping everthing in one single textbox randomly from config file)
>
> SMTP
> - config1=value1
> - config2=value2
> - config3=value3 ...
> AUTHENTICATION
> - KERBERSOE
> - config1=value1
> - config2=value2
> - config3=value3 ...
> - LDAP
> - config1=value1
> - config2=value2
> - config3=value3 ...
> SECURITY
> - config1=value1
> - config2=value2
> - config3=value3 ...
>

It would, but it's not really worth the effort. This is really intended as
debugging info the user can just copy/paste into an email to us.

>
>
> Regards,
> Murtuza
>
> On Fri, Jun 4, 2021 at 5:33 PM Pradip Parkale <
> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>
>> Hi Akshay,
>>
>> Please find the updated patch. I have fixed the OS details issue for
>> Linux, MacOS big sur.
>>
>> On Tue, Jun 1, 2021 at 8:48 PM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Thanks, patch applied.
>>>
>>> On Tue, Jun 1, 2021 at 2:03 PM Pradip Parkale <
>>> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Akshay,
>>>>
>>>> Please find an updated patch.
>>>>
>>>>
>>>> On Mon, May 24, 2021 at 4:45 PM Akshay Joshi <
>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Pradip
>>>>>
>>>>> Following are the review comments:
>>>>>
>>>>> - Following labels should be changed
>>>>> - Browser Details -> Browser
>>>>> - OS -> Operating System
>>>>> - Config DB -> pgAdmin Database File
>>>>>
>>>>> Done.
>>>>
>>>>>
>>>>> - Server Configuration Text Area should be *readonly* and it
>>>>> should have a "col-sm-9" class instead of "col-sm-3".
>>>>>
>>>>> Done.
>>>>
>>>>>
>>>>> - If the user resizes the About dialog then instead of showing
>>>>> blank space can we automatically resize the Server configuration text area?
>>>>>
>>>>> Done.
>>>>
>>>>>
>>>>> - The height of the About dialog in the Server mode should be less
>>>>> as we are not showing "Server Configuration".
>>>>>
>>>>> Done.
>>>>
>>>>>
>>>>> - In Runtime, we should merge the 'Browser details' in 'NW.js
>>>>> version' like *0.51.2 (Chromium 89.0.4389.114)* OR at least prefix
>>>>> 'Chromium' before the version in the 'Browser details'.
>>>>>
>>>>> Done.
>>>>
>>>>>
>>>>> - Found string difference in OS details for Runtime and Desktop
>>>>> mode.
>>>>> - OSX: "Intel Mac OS X 10_15_7" in Runtime, while "Mac OS X
>>>>> 10.15.7" in Dekstop mode.
>>>>> - Windows: "Win64" in Runtime, while "Windows 10" in Desktop
>>>>> which also not correct, I am using "Windows Server 2016".
>>>>> - Following issue found on Firefox (First Image) configuration in
>>>>> a single line, Safari (Second image) too many scroll bars.
>>>>>
>>>>> Done.
>>>>
>>>>> [image: Firefox.png]. [image: Safari.png]
>>>>>
>>>>> On Fri, May 21, 2021 at 2:23 PM Pradip Parkale <
>>>>> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi Hackers,
>>>>>>
>>>>>> Please find the attached patch for #6231. I have added OS, NW.js,
>>>>>> browser details, and some server configuration in the About pgAdmin pop-up.
>>>>>>
>>>>>> Server configuration won't be visible to the non-admin users in
>>>>>> server mode.
>>>>>> --
>>>>>> Thanks & Regards,
>>>>>> Pradip Parkale
>>>>>> Software Engineer | EnterpriseDB Corporation
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Thanks & Regards*
>>>>> *Akshay Joshi*
>>>>> *pgAdmin Hacker | Principal Software Architect*
>>>>> *EDB Postgres <http://edbpostgres.com>*
>>>>>
>>>>> *Mobile: +91 976-788-8246*
>>>>>
>>>>
>>>>
>>>> --
>>>> Thanks & Regards,
>>>> Pradip Parkale
>>>> Software Engineer | EnterpriseDB Corporation
>>>>
>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>> *pgAdmin Hacker | Principal Software Architect*
>>> *EDB Postgres <http://edbpostgres.com>*
>>>
>>> *Mobile: +91 976-788-8246*
>>>
>>
>>
>> --
>> Thanks & Regards,
>> Pradip Parkale
>> Software Engineer | EnterpriseDB Corporation
>>
>

--
Dave Page
VP, Chief Architect, Database Infrastructure
Blog: https://www.enterprisedb.com/dave-page
Twitter: @pgsnake

EDB: https://www.enterprisedb.com