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

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
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-06 07:55:47
Message-ID: CA+OCxozxbKf2FhVjMZCRdjETDWACoEMV7cpGPsW0jcumLrkNSQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Fri, Apr 6, 2018 at 8:52 AM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> Hi Dave,
>
> I checked and found that the issue was in wcDocker which was causing
> problem while opening query tool,
> I have logged an issue on github: https://github.com/
> WebCabin/wcDocker/issues/124
>
> Query tool, Debugger etc are working properly when open in new Browser Tab
> and not in Panel.
>
> I'm also able to fix the issue which I was facing while running Jasmine
> test by correctting the path for *jquery.event.drag* library from
> jquery.event.drag-2.2 to jquery.event.drag-2.3.0.
>
> Let's wait for author's reply on the issue.
>

Great, thanks.

>
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> On Wed, Apr 4, 2018 at 3:20 PM, Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.com> wrote:
>
>> 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(dot)zabuawala(at)enterprisedb(dot)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
>>>
>>
>>
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2018-04-06 08:48:21 Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree
Previous Message Murtuza Zabuawala 2018-04-06 07:52:15 Re: [pgAdmin4][RM#3154] Update modules to latest version