From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4] [Patch]: Language Module |
Date: | 2016-02-22 11:59:51 |
Message-ID: | CA+OCxozm4WGF9mJ=SWX5o185yY2Gm0q8Y_5XkX4ovsPBU6zaLg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Mon, Feb 22, 2016 at 10:53 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>
> On Fri, Feb 19, 2016 at 10:48 AM, Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Hi All
>>
>> On Mon, Feb 15, 2016 at 3:47 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>>
>>>
>>> On Mon, Feb 15, 2016 at 7:29 AM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi All
>>>>
>>>> Attached is the new patch after fixing review comments given by Dave.
>>>>
>>>> On Fri, Feb 12, 2016 at 10:04 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Fri, Feb 12, 2016 at 11:21 AM, Harshal Dhumal <
>>>>> harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> New language patch is working fine.
>>>>>> Ashesh - Can you please review it ?
>>>>>> If it looks good then can you please commit ?
>>>>>>
>>>>>> Thank you,
>>>>>> Harshal
>>>>>>
>>>>>
>>>>> A few comments:
>>>>>
>>>>> - We omitted CREATE LANGUAGE, as most are now extensions. Shouldn't we
>>>>> also therefore omit DROP LANGUAGE?
>>>>>
>>>> Fixed.
>>>>
>>>>> - Please leave a blank line between the file header and python code in
>>>>> python files.
>>>>>
>>>> Fixed.
>>>>
>>>>> - The pydoc module description is missing from __init__.py
>>>>>
>>>> Fixed.
>>>>
>>>>> - What about the Dependents and Dependencies tabs?
>>>>>
>>>> Infrastructure is not ready, so not implemented yet.
>>>>
>>>
>>> The first person that needs it should be working on it! I don't want to
>>> be committing partially completed features, otherwise we'll have to come
>>> back and revisit them all when the infrastructure is done.
>>>
>>> Having done it before for pgAdmin 3, I don't imagine it's hard. It's
>>> really just the same as the Statistics view. Please take a look at it
>>> (assuming noone else is already).
>>>
>>
>> Fixed. Attached is the modified patch with support of Dependents and
>> Dependencies tabs. Please review it.
>>
>
> Committed, with a couple of additional minor changes:
>
> - s/resourcegroups/resource_groups (for consistency with server_groups)
>
> - Added a PyDoc module description to __init__.py
>
Oops, wrong thread. Please ignore.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2016-02-22 12:00:34 | Re: [pgAdmin4] [Patch]: Resource Group Module |
Previous Message | Ashesh Vashi | 2016-02-22 11:50:50 | Re: [pgAdmin4] Numeric Control for Backform |