Re: [pgAdmin4][RM#3238] Proper error handling when connection will be lost to server.

From: Anthony Emengo <aemengo(at)pivotal(dot)io>
To: aditya(dot)toshniwal(at)enterprisedb(dot)com
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][RM#3238] Proper error handling when connection will be lost to server.
Date: 2018-05-03 12:23:01
Message-ID: CAG8BBZOLBWghk=y0=GcgV20QAV=56q6-3ima+SPbzHD5LEDWVw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

In addition, please address the following linting issues:

*Python:*
[image: Screen Shot 2018-05-03 at 8.18.29 AM.png]

*Javascript:*
[image: Screen Shot 2018-05-03 at 8.22.11 AM.png]

On Thu, May 3, 2018 at 6:38 AM Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Thu, May 3, 2018 at 11:23 AM, Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> PFA patch for proper error handling when connection will be lost to
>> server. I have created a common function alertify.pgRespErrorNotify in
>> web/pgadmin/static/js/alertify.pgadmin.defaults.js on the lines of existing
>> function alertify.pgNotifier. The function is called in error callback of
>> $.ajax call. The XHR is passed and handled in the function itself. Please
>> note that I have used JSON.parse and not $.parseJSON as it will be
>> deprecated in jQuery 3.x and thus removing many $.parseJSON calls.
>>
>> Request you to kindly review.
>>
>
> Can you add some appropriate tests for the new function please?
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Anthony Emengo 2018-05-03 14:23:21 [pgAdmin4][patch] - Fix CheckForViewDataTest
Previous Message Dave Page 2018-05-03 10:38:05 Re: [pgAdmin4][RM#3238] Proper error handling when connection will be lost to server.