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

From Ashesh Vashi
Subject Re: PATCH: To fix the issue in type node (pgAdmin4)
Date
Msg-id CAG7mmoyFu2iRPeNE_OdnDBM9GQm_3Ay6xi2c4A3zLZqRYaQ85w@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: To fix the issue in type node (pgAdmin4)  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers
Thanks - committed!

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


http://www.linkedin.com/in/asheshvashi


On Thu, Jun 2, 2016 at 4:03 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
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: pgAdmin 4 commit: Fixes #1273 - Resolved an issue setting privileges on
Next
From: Harshal Dhumal
Date:
Subject: Destroy header controls properly [pgadmin4]