Re: [pgAdmin4][RM#3238] Proper error handling when connection will be lost to server. - Mailing list pgadmin-hackers

From Anthony Emengo
Subject Re: [pgAdmin4][RM#3238] Proper error handling when connection will be lost to server.
Date
Msg-id CAG8BBZOLBWghk=y0=GcgV20QAV=56q6-3ima+SPbzHD5LEDWVw@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][RM#3238] Proper error handling when connection will belost to server.  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][RM#3238] Proper error handling when connection will be lost to server.  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
List pgadmin-hackers
In addition, please address the following linting issues:

Python:
Screen Shot 2018-05-03 at 8.18.29 AM.png

Javascript:
Screen Shot 2018-05-03 at 8.22.11 AM.png


On Thu, May 3, 2018 at 6:38 AM Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, May 3, 2018 at 11:23 AM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.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
Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4][RM#3238] Proper error handling when connection will belost to server.
Next
From: Anthony Emengo
Date:
Subject: [pgAdmin4][patch] - Fix CheckForViewDataTest