Re: [pgAdmin4][Patch] - RM 5422 - Synonym Dependencies tab shows incorrect data

2020-05-04 Thread Akshay Joshi
Thanks, patch applied.

On Wed, Apr 29, 2020 at 1:29 PM Khushboo Vashi <
khushboo.va...@enterprisedb.com> wrote:

> Hi Navnath,
>
> On Wed, Apr 29, 2020 at 12:16 PM navnath gadakh <
> navnath.gad...@enterprisedb.com> wrote:
>
>> Hi Khushboo,
>>
>>  Patch looks good to me except there are no API test cases written for
>> dependents and dependencies.
>>
>> Thanks for the review.
> We don't have API test cases for dependents and dependencies for none of
> the objects.
> If the team agrees, I will create a separate RM for that.
>
> Thanks,
> Khushboo
>
>> Thanks!
>>
>> On Wed, Apr 29, 2020 at 10:40 AM navnath gadakh <
>> navnath.gad...@enterprisedb.com> wrote:
>>
>>> Hi ,
>>>
>>>  I'm reviewing this task.
>>>
>>> On Wed, Apr 29, 2020 at 9:36 AM Khushboo Vashi <
>>> khushboo.va...@enterprisedb.com> wrote:
>>>
 Hi,

 Please find the attached patch to fix the RM 5422 - Synonym
 Dependencies tab shows incorrect data.

 Thanks,
 Khushboo

>>>
>>>
>>> --
>>> Regards,
>>> Navnath Gadakh
>>>
>>
>>
>> --
>> Regards,
>> Navnath Gadakh
>>
>

-- 
*Thanks & Regards*
*Akshay Joshi*

*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*


Re: [pgAdmin4][Patch] - RM 5422 - Synonym Dependencies tab shows incorrect data

2020-04-29 Thread Khushboo Vashi
Hi Navnath,

On Wed, Apr 29, 2020 at 12:16 PM navnath gadakh <
navnath.gad...@enterprisedb.com> wrote:

> Hi Khushboo,
>
>  Patch looks good to me except there are no API test cases written for
> dependents and dependencies.
>
> Thanks for the review.
We don't have API test cases for dependents and dependencies for none of
the objects.
If the team agrees, I will create a separate RM for that.

Thanks,
Khushboo

> Thanks!
>
> On Wed, Apr 29, 2020 at 10:40 AM navnath gadakh <
> navnath.gad...@enterprisedb.com> wrote:
>
>> Hi ,
>>
>>  I'm reviewing this task.
>>
>> On Wed, Apr 29, 2020 at 9:36 AM Khushboo Vashi <
>> khushboo.va...@enterprisedb.com> wrote:
>>
>>> Hi,
>>>
>>> Please find the attached patch to fix the RM 5422 - Synonym Dependencies
>>> tab shows incorrect data.
>>>
>>> Thanks,
>>> Khushboo
>>>
>>
>>
>> --
>> Regards,
>> Navnath Gadakh
>>
>
>
> --
> Regards,
> Navnath Gadakh
>


Re: [pgAdmin4][Patch] - RM 5422 - Synonym Dependencies tab shows incorrect data

2020-04-29 Thread navnath gadakh
Hi Khushboo,

 Patch looks good to me except there are no API test cases written for
dependents and dependencies.

Thanks!

On Wed, Apr 29, 2020 at 10:40 AM navnath gadakh <
navnath.gad...@enterprisedb.com> wrote:

> Hi ,
>
>  I'm reviewing this task.
>
> On Wed, Apr 29, 2020 at 9:36 AM Khushboo Vashi <
> khushboo.va...@enterprisedb.com> wrote:
>
>> Hi,
>>
>> Please find the attached patch to fix the RM 5422 - Synonym Dependencies
>> tab shows incorrect data.
>>
>> Thanks,
>> Khushboo
>>
>
>
> --
> Regards,
> Navnath Gadakh
>


-- 
Regards,
Navnath Gadakh


Re: [pgAdmin4][Patch] - RM 5422 - Synonym Dependencies tab shows incorrect data

2020-04-28 Thread navnath gadakh
Hi ,

 I'm reviewing this task.

On Wed, Apr 29, 2020 at 9:36 AM Khushboo Vashi <
khushboo.va...@enterprisedb.com> wrote:

> Hi,
>
> Please find the attached patch to fix the RM 5422 - Synonym Dependencies
> tab shows incorrect data.
>
> Thanks,
> Khushboo
>


-- 
Regards,
Navnath Gadakh