Re: [pgAdmin4][RM#3154] Update modules to latest version

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][RM#3154] Update modules to latest version
Date: 2018-04-04 09:50:44
Message-ID: CAKKotZQsx8MQ1Di+bm9t5aeWXU-kMeYQKi-rrfntjqF+7mNR+g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Wed, Apr 4, 2018 at 3:19 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Wed, Apr 4, 2018 at 8:24 AM, Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.com> wrote:
>
>> Hi Joao,
>>
>> Yes it bundles latest version but I doubt that is not using it, If we
>> upgrade the jQuery to 3.3.1 then it's breaking the SlickGrid.
>> I have tried that and I started getting jQuery related errors when trying
>> to run query tool or when running jasmine tests.
>>
>
> Can you spend some time trying to figure it out please? We need to get on
> to 3.x as older versions are no longer receiving updates which could pose a
> security risk.
>
​Sure Dave.​

>
>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>>
>> On Wed, Apr 4, 2018 at 1:39 AM, Joao De Almeida Pereira <
>> jdealmeidapereira(at)pivotal(dot)io> wrote:
>>
>>> Hi Murtuza,
>>>
>>> The patches look good and they pass all tests in CI.
>>> One think that we realized was the SlickGrid as a npm package now:
>>> https://www.npmjs.com/package/slickgrid
>>> Also Slickgrid comes packaged with jquery 3.1 not sure if it is fully
>>> supported or not......
>>>
>>> Thanks
>>> Victoria & Joao
>>>
>>> On Tue, Apr 3, 2018 at 11:50 AM Murtuza Zabuawala <
>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>
>>>> ​Hi Dave,
>>>>
>>>> Reverted back jQuery version to 1.x because of SlickGrid dependancy.
>>>> Please find updated patch.
>>>>
>>>> --
>>>> Regards,
>>>> Murtuza Zabuawala
>>>> EnterpriseDB: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>>
>>>> On Tue, Apr 3, 2018 at 8:50 PM, Murtuza Zabuawala <
>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Please hold on my previous patch.
>>>>>
>>>>> We can't use latest jQuery version as SlickGrid has dependancy on
>>>>> older version.
>>>>> I'll send updated patch again.
>>>>>
>>>>> On Tue, Apr 3, 2018 at 8:20 PM, Murtuza Zabuawala <
>>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi Dave,
>>>>>>
>>>>>> Please find updated patches, there are two patches attached one is
>>>>>> for the story and another is for changes required as per new modules.
>>>>>>
>>>>>>
>>>>>> On Tue, Apr 3, 2018 at 6:27 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>>
>>>>>>> Hi
>>>>>>>
>>>>>>> On Fri, Mar 30, 2018 at 8:01 AM, Murtuza Zabuawala <
>>>>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>>>>
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> PFA patch to update the modules to latest version.
>>>>>>>>
>>>>>>>>
>>>>>>> - Why isn't jQuery updated to 3.3.1?
>>>>>>>
>>>>>> ​Done​
>>>>>>
>>>>>>
>>>>>>> ​
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>> - Shouldn't pkg/pip/setup_pip.py be updated with changes to psycopg2
>>>>>>> and pycrypto etc?
>>>>>>>
>>>>>> ​Done​
>>>>>>
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>> We are not able to update some of the modules to latest version due
>>>>>>>> to dependancy on other modules, For example
>>>>>>>> - Python: Flask-Security has dependancy on flask-babelex which
>>>>>>>> causes conflict with flask_babel
>>>>>>>>
>>>>>>>
>>>>>>> Hmm, flask-babelex might be a better option anyway; in particular,
>>>>>>> it avoids loading catalogs with every request which seems desirable given
>>>>>>> the size of ours. On the other hand, it hasn't been updated so recently.
>>>>>>>
>>>>>> ​I have removed flask-babel and used flask_babelex instead.​
>>>>>>
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>> - JS: Can't update to Bootstrap4 because Bootstrap Switch &
>>>>>>>> Bootstrap Datetime picker has dependancy on Bootstrap3.
>>>>>>>>
>>>>>>>
>>>>>>> OK.
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Dave Page
>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>> Twitter: @pgsnake
>>>>>>>
>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-04-04 10:20:39 pgAdmin 4 commit: Refactor and simplify query tool connection error han
Previous Message Dave Page 2018-04-04 09:49:28 Re: [pgAdmin4][RM#3154] Update modules to latest version