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

Hi

On Fri, Jun 4, 2021 at 2:34 PM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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.parkale@enterprisedb.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.joshi@enterprisedb.com> wrote:

Thanks, patch applied.

On Tue, Jun 1, 2021 at 2:03 PM Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:

Hi Akshay,

Please find an updated patch.

On Mon, May 24, 2021 at 4:45 PM Akshay Joshi <akshay.joshi@enterprisedb.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. 

                  .      

On Fri, May 21, 2021 at 2:23 PM Pradip Parkale <pradip.parkale@enterprisedb.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

Mobile: +91 976-788-8246

--

Thanks & Regards,

Pradip Parkale
Software Engineer | EnterpriseDB Corporation

--

Thanks & Regards

Akshay Joshi

pgAdmin Hacker | Principal Software Architect

EDB Postgres

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