Thanks Neel for reviewing. I'll send the patch with the fixes suggested.

On Mon, Jan 18, 2016 at 4:52 PM, Neel Patel <neel.pa...@enterprisedb.com>
wrote:

> Hi Surinder,
>
> We have applied/tested the patch and below are the review comments.
>
> 1. When we select the extension "plpython3u", "plperl", "plperu" etc. then
> it gives 'TypeError' in Javascript.
> TypeError: d.version is undefined
> 'version': (!_.isNull(d.version[0]) ? d.version[0]: '')
>
> We are getting this error while selecting many extensions so please test
> with all types of extensions, it should not give any error at client side.
>
> 2.  Use 2 space indentation instead of 4 space in javascript file.
>
> 3. In "validate" function in "extension.js" file, validate only the
> changed values not all, and "this.get('name') - should be called only one
> time not multiple
>     time".
>
> 4. When we pass object identifier, use the function 'qtIdent', and for
> the values, use function  'qtLiteral' in all the sql files.
>
> 5. By default, when we create the extension, "schema_name" and "version"
> should not be be set with value. It should be set blank by default.
>
> 6. When we create any extension like "citext" then we are not able to
> create the same extension again after deleting the same extension. May be
> issue
>     with caching mechanism.
>
> 7. When we remove the schema_name during the "Edit" operation then wrong
> SQL is getting generated.
>
> 8. Remove "Use Slony" option. As discussed with Ashesh, we will implement
> it as separate module.
>
>
> Please fix the above issues. Let us know if you want more information.
>
> Thanks,
> Neel Patel
>
>
> Thanks,
> Neel Patel
>
>
> On Tue, Jan 12, 2016 at 1:15 PM, Surinder Kumar <
> surinder.ku...@enterprisedb.com> wrote:
>
>> Please find the updated patch with following changes:
>>
>>    1. corrected copyright.
>>    2. Added proper comment for script_module function in __init__.py
>>    file.
>>    3. Renamed collection Node's label to Extensions in extensions.js
>>    file.
>>
>>
>> On Tue, Jan 12, 2016 at 12:44 PM, Surinder Kumar <
>> surinder.ku...@enterprisedb.com> wrote:
>>
>>> Hi,
>>>
>>> Please find attached patch for the extension module.
>>> Please review it and Let me know for any comments.
>>>
>>>
>>> Thanks,
>>> Surinder Kumar
>>>
>>
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>>
>

Reply via email to