Re: PATCH: Select2 control fix [ pgAdmin4 ]

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: Select2 control fix [ pgAdmin4 ]
Date: 2016-01-12 09:00:56
Message-ID: CAG7mmoyZvKw0wcwgv_rdp=-1f8fhZ64-yiKuoZ3Nw=nwQgX2yA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Murtuza,

I have applied the patch manually, and committed.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
<http://www.enterprisedb.com>

*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>

On Tue, Jan 12, 2016 at 10:56 AM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> Hi Ashesh,
>
> Can you please try applying given updated patch, I have manually converted
> file line ending to Unix format.
>
>
> Regards,
> Murtuza
>
>
> On Tuesday 12 January 2016 10:33 AM, Murtuza Zabuawala wrote:
>
> Hi Ashesh,
>
> I am not sure what's wrong here as I'm able to apply the given patch with
> latest pull.
>
> murtuza(at)debian ~/PROJECT/pgadmin4 $ git pull
> Already up-to-date.
> murtuza(at)debian ~/PROJECT/pgadmin4 $ patch -p1 <
> Fix_select2_disabled_property.patch
> patching file web/pgadmin/browser/static/js/node.ui.js
> murtuza(at)debian ~/PROJECT/pgadmin4 $ git diff
> diff --git a/web/pgadmin/browser/static/js/node.ui.js
> b/web/pgadmin/browser/static/js/node.ui.js
> index 76839c6..b5fa3e2 100644
>
>
> And here are my git settings,
>
> murtuza(at)debian ~/PROJECT/pgadmin4 $ git config -l
> core.editor=vim
> core.autocrlf=input
> core.repositoryformatversion=0
> core.filemode=true
> core.bare=false
> core.logallrefupdates=true
> remote.origin.url=http://git.postgresql.org/git/pgadmin4.git
> remote.origin.fetch=+refs/heads/*:refs/remotes/origin/*
> branch.master.remote=origin
> branch.master.merge=refs/heads/master
>
>
> Regards,
> Murtuza
>
>
>
> On Monday 11 January 2016 09:36 PM, Ashesh Vashi wrote:
>
> Hi Murtuza,
>
> I am not able to apply the patch.
> Can you please rebase the patch?
>
> (Also - please set the proper line endings in your git repository.)
>
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company
> <http://www.enterprisedb.com>
>
>
> *http://www.linkedin.com/in/asheshvashi*
> <http://www.linkedin.com/in/asheshvashi>
>
> On Mon, Jan 11, 2016 at 5:48 PM, Murtuza Zabuawala <
> <murtuza(dot)zabuawala(at)enterprisedb(dot)com>murtuza(dot)zabuawala(at)enterprisedb(dot)com>
> wrote:
>
>> Hi,
>>
>> Please find minor patch to fix issue in select2 control.
>>
>> Issue: We were not able to apply 'disabled' property from schema model on
>> select2 control.
>>
>>
>> Regards,
>> Murtuza
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list ( <pgadmin-hackers(at)postgresql(dot)org>
>> pgadmin-hackers(at)postgresql(dot)org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>>
>
>
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2016-01-12 09:42:19 Re: pgAdmin 4 commit: Update the user and datbaase information everytime, w
Previous Message Ashesh Vashi 2016-01-12 08:11:37 pgAdmin 4 commit: Fixes few UI issue with the backform controls.