From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | Joao Pedro De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Matthew Kleiman <mkleiman(at)pivotal(dot)io> |
Subject: | Re: [pgAdmin4][PATCH] To fix the issue of menu visibility when node is hidden |
Date: | 2017-05-12 11:11:43 |
Message-ID: | CA+OCxoy4w_gEacgU40-KdcrS7vs0d2-7wfkcjDxDNbrgxSdU1g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, applied.
On Fri, May 12, 2017 at 11:59 AM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
> Hi,
>
> Please find updated patch with changes as per comments given by Joao.
>
> @Joao,
> Thank you for reviewing the patch.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> On Thu, May 11, 2017 at 8:19 PM, Joao Pedro De Almeida Pereira <
> jdealmeidapereira(at)pivotal(dot)io> wrote:
>
>> Hi Murtuza,
>>
>> Since you are adding a new function to the browser.js component, we
>> would encourage you to extract the function out of the templated javascript
>> file so that it can be tested. A good example of this would be the
>> pgadmin/static/js/size_prettify.js.
>>
>> For this specific function we found one function that is used from
>> browser.js, and you can Stub that out using
>> createSpyObject(browser, 'get_preference');
>>
>> We think the object here is not 100% line coverage but 100% of the
>> behavior should be covered.
>>
>> Also, suggest you change the variable name perf to preference. Looks
>> like it is a typo.
>>
>> Regards,
>> João & Matt
>>
>> On Thu, May 11, 2017 at 3:08 AM, Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> PFA patch to fix the issue where user hides the any node from Preference
>>> dialog but the menu(both context/object) still appears in pgAdmin4.
>>> RM#2225
>>>
>>> --
>>> 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
>>>
>>>
>>
>
>
> --
> 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
>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Harshal Dhumal | 2017-05-12 11:19:55 | Re: pgAdmin 4 commit: Improve handling of nulls and default values in the d |
Previous Message | Dave Page | 2017-05-12 11:10:53 | pgAdmin 4 commit: Hide menu options for creating objects, if the object |