From: | Priyanka Shendge <priyanka(dot)shendge(at)enterprisedb(dot)com> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Synonym node for EPAS (pgAdmin4) |
Date: | 2016-08-26 11:14:57 |
Message-ID: | CAKmZXFS9j0gL5YG3qMFz2D8NHorJKEfG--HMcTW+Y9=cVDjKvA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Murtuza,
Please find my review comments below:
1. Please attach test_advanced_config.json.in file to read the config data
2. While reading data from config file "owner" field is missing
3. Please add synonym pickle object to ../regression/test_utils.py file
4. Can we have config data parameters generated dynamically? (e.g. schema
name)
Let me know for any assistance.
Thank you.
On 26 August 2016 at 13:03, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Thanks, patch applied.
>
> On Thu, Aug 25, 2016 at 7:58 PM, Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.com> wrote:
>
>> Hi,
>>
>> Please find updated patch.
>>
>> I have also fixed the issue in package node which was displaying create
>> menu in PG server. (separate patch attached)
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>> On Thu, Aug 25, 2016 at 4:34 PM, Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Murtuza
>>>
>>> Following are my review comments:
>>>
>>> - Create "Synonym" option is visible for Postgres Servers, when
>>> right click on schema node.
>>>
>>> Fixed
>>
>>>
>>> - Unable to create Public Synonym faced "Internal Server Error".
>>>
>>> Fixed
>>
>>>
>>> - SQL help not working when click on "[i]" button.
>>>
>>> Fixed, Now it is disbaled
>>
>>>
>>> - "Public Synonyms?" switch control showing True/False when click on
>>> Synonyms collection node and it is showing Yes/No when click on synonym
>>> node under properties tab.
>>>
>>> Fixed
>>
>>>
>>> On Thu, Aug 25, 2016 at 3:08 PM, Murtuza Zabuawala <
>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> PFA updated patch with `--binary` option for .png files.
>>>>
>>>> --
>>>> Regards,
>>>> Murtuza Zabuawala
>>>> EnterpriseDB: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>> On Thu, Aug 25, 2016 at 12:27 PM, Murtuza Zabuawala <
>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> PFA patch for synonym node, this patch also includes basic unit test
>>>>> cases.
>>>>> Please review.
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Murtuza Zabuawala
>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>>>> To make changes to your subscription:
>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>
>>>>
>>>
>>>
>>> --
>>> *Akshay Joshi*
>>> *Principal Software Engineer *
>>>
>>>
>>>
>>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>>
>>
>>
>
>
> --
> *Akshay Joshi*
> *Principal Software Engineer *
>
>
>
> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>
--
Best,
Priyanka
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Priyanka Shendge | 2016-08-26 11:28:47 | Re: PATCH: Synonym node for EPAS (pgAdmin4) |
Previous Message | Akshay Joshi | 2016-08-26 09:55:52 | pgAdmin 4 commit: Getting 'AttributeError' in pgadmin4/web/pgadmin/brow |