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

From Yogesh Mahajan
Subject Re: [pgAdmin4][Patch] - RM #5465 - [Microsoft Edge] Mensagem de Erro Indevida
Date
Msg-id CAMa=N=MXt_Nq+n1e2R1rBW4S6WSwVKbVxpK7-rrQmHkSjXguEg@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch] - RM #5465 - [Microsoft Edge] Mensagem de Erro Indevida  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Responses Re: [pgAdmin4][Patch] - RM #5465 - [Microsoft Edge] Mensagem de Erro Indevida  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
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"
Attachment

pgadmin-hackers by date:

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