Re: [pgAdmin4][Patch] - RM #5465 - [Microsoft Edge] Mensagem de Erro Indevida - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [pgAdmin4][Patch] - RM #5465 - [Microsoft Edge] Mensagem de Erro Indevida
Date
Msg-id CANxoLDfTSLiNtswvxqpr-3wr3xjSE1-4AaD5nmv3-qtiu8bWbw@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch] - RM #5465 - [Microsoft Edge] Mensagem de Erro Indevida  (Yogesh Mahajan <yogesh.mahajan@enterprisedb.com>)
List pgadmin-hackers
Thanks, patch applied.

On Fri, May 29, 2020 at 12:17 PM Yogesh Mahajan <yogesh.mahajan@enterprisedb.com> wrote:
Hi Hackers,

Comparing with 18 also will create the same issue as there are multiple Edge browser versions mapped to single EdgeHTML engine versions.
Please find modified patch which will show exact detected browser version till Edge Version 40. For  Edge version < 40, error message will show  version '<44'.



Thanks,
Yogesh Mahajan
QA - Team
EnterpriseDB Corporation

Phone: +91-9741705709


On Fri, May 29, 2020 at 11:19 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Yogesh,

Your patch will fail if the edge version is older than 39. I would suggest comparing version > 18 directly.

On Fri, May 29, 2020 at 10:57 AM Yogesh Mahajan <yogesh.mahajan@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch which shows the actual Edge browser version when pgAdmin is launched Edge Version <43.

Thanks,
Yogesh Mahajan
QA - Team
EnterpriseDB Corporation

Phone: +91-9741705709


--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed an issue where pgadmin detects the wrong browse
Next
From: Dave Page
Date:
Subject: Re: Windows build system cleanup