Re: PATCH: To fix the issue in type node (pgAdmin4)

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(dot)vashi(at)enterprisedb(dot)com> wrote:
>
> On Thu, Jun 2, 2016 at 3:53 PM, Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com <mailto:murtuza(dot)zabuawala(at)enterprisedb(dot)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(dot)vashi(at)enterprisedb(dot)com <mailto:ashesh(dot)vashi(at)enterprisedb(dot)com>> wrote:
>>
>> On Wed, Jun 1, 2016 at 6:25 PM, Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com <mailto:murtuza(dot)zabuawala(at)enterprisedb(dot)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 <http://www.enterprisedb.com/>
>>
>> http://www.linkedin.com/in/asheshvashi <http://www.linkedin.com/in/asheshvashi>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com <http://www.enterprisedb.com/>
>> The Enterprise PostgreSQL Company
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org <mailto:pgadmin-hackers(at)postgresql(dot)org>)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers <http://www.postgresql.org/mailpref/pgadmin-hackers>
>>
>>
>
>