Re: PATCH: To fix the issue in type node (pgAdmin4) - Mailing list pgadmin-hackers

From Murtuza Zabuawala
Subject Re: PATCH: To fix the issue in type node (pgAdmin4)
Date
Msg-id AED5DA0A-5B75-4574-BEF7-7290B2273B2D@enterprisedb.com
Whole thread Raw
In response to Re: PATCH: To fix the issue in type node (pgAdmin4)  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Responses Re: PATCH: To fix the issue in type node (pgAdmin4)
List pgadmin-hackers
Hi,

We had condition where we were validating some model objects only if it’s in sessAttrs, which I have removed it now.
Issue: we were not getting error for required fields.


Regards,
Murtuza

On 02-Jun-2016, at 3:55 pm, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Thu, Jun 2, 2016 at 3:53 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Ashesh,

I have fixed the issue and also corrected some validations as well in privileges.js where we were using object before validation it.
Can you please give some details about those validations?

What was the issue, and how did you fix it?
 

Thanks,
Murtuza.
 
On 02-Jun-2016, at 3:40 pm, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Wed, Jun 1, 2016 at 6:25 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch to fix the issue when throwing "Internal server error" on adding Privileges
(RM#1273)

The patch contains multiple fixes in the Type node.
Can you please explain, what these changes are for?

--
Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers





pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: PATCH: To fix the issue in type node (pgAdmin4)
Next
From: Murtuza Zabuawala
Date:
Subject: Re: PATCH: Fix the issue in JSON decoding (pgAdmin4)