From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | John Obaterspok <john(dot)obaterspok(at)gmail(dot)com>, Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Add Commit/Rollback toolbar action version 2 |
Date: | 2015-09-25 10:21:26 |
Message-ID: | CAG7mmoxZqS_b-jokYTf3FXsvK-b89Me33kouG4-AFsvunoCzxg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Fri, Sep 25, 2015 at 1:30 PM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com
> wrote:
> Yup.
> It's in my plate, was updating the CHANGELOG only.
>
Done.
Thanks John for the patch, and Sanket for reviewing the patch.
--
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>
>
> --
>
> 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 Fri, Sep 25, 2015 at 1:28 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Ashesh, are you able to deal with this before Monday (when I'll be
>> cutting the feature-frozen beta 1 tarballs)?
>>
>> On Fri, Sep 25, 2015 at 7:03 AM, John Obaterspok <
>> john(dot)obaterspok(at)gmail(dot)com> wrote:
>>
>>> Thanks, appreciated.
>>>
>>> Can it be committed then? :)
>>>
>>> 2015-09-23 9:49 GMT+02:00 Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>:
>>>
>>>> Hi John/Ashesh,
>>>>
>>>> I have reviewed the patch and it seems fine to me.
>>>>
>>>> Regards,
>>>> Sanket Mehta
>>>> Sr Software engineer
>>>> Enterprisedb
>>>>
>>>> On Wed, Sep 23, 2015 at 11:08 AM, John Obaterspok <
>>>> john(dot)obaterspok(at)gmail(dot)com> wrote:
>>>>
>>>>> Hi Sanket,
>>>>>
>>>>> Did you give the patch a try? I would really like it to for the next
>>>>> release :)
>>>>>
>>>>> -- john
>>>>>
>>>>> 2015-09-16 20:27 GMT+02:00 John Obaterspok <john(dot)obaterspok(at)gmail(dot)com>
>>>>> :
>>>>>
>>>>>> Hi Sanket,
>>>>>>
>>>>>> Attached patch is rebased now.
>>>>>>
>>>>>> Regards,
>>>>>>
>>>>>> -- john
>>>>>>
>>>>>>
>>>>>> 2015-09-16 11:20 GMT+02:00 Sanket Mehta <
>>>>>> sanket(dot)mehta(at)enterprisedb(dot)com>:
>>>>>>
>>>>>>> Hi John,
>>>>>>>
>>>>>>> Patch is not proper.
>>>>>>> I got below error when I try to apply the patch:
>>>>>>>
>>>>>>> error: patch failed: pgadmin/frm/frmQuery.cpp:2435
>>>>>>> error: pgadmin/frm/frmQuery.cpp: patch does not apply
>>>>>>>
>>>>>>> Please do provide the proper patch.
>>>>>>>
>>>>>>>
>>>>>>> Regards,
>>>>>>> Sanket Mehta
>>>>>>> Sr Software engineer
>>>>>>> Enterprisedb
>>>>>>>
>>>>>>> On Mon, Sep 14, 2015 at 10:51 AM, Sanket Mehta <
>>>>>>> sanket(dot)mehta(at)enterprisedb(dot)com> wrote:
>>>>>>>
>>>>>>>> Sure Ashesh,
>>>>>>>>
>>>>>>>> I will review the patch.
>>>>>>>>
>>>>>>>> Regards,
>>>>>>>> Sanket Mehta
>>>>>>>> Sr Software engineer
>>>>>>>> Enterprisedb
>>>>>>>>
>>>>>>>> On Mon, Sep 14, 2015 at 1:12 PM, Ashesh Vashi <
>>>>>>>> ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>>>>>>
>>>>>>>>> Sanket,
>>>>>>>>>
>>>>>>>>> Can you please take a look at it?
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>>
>>>>>>>>> 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, Sep 14, 2015 at 1:07 AM, John Obaterspok <
>>>>>>>>> john(dot)obaterspok(at)gmail(dot)com> wrote:
>>>>>>>>>
>>>>>>>>>> Hello,
>>>>>>>>>>
>>>>>>>>>> I'm updated the patch to add commit/rollback actions in the
>>>>>>>>>> toolbar with the following:
>>>>>>>>>> 1) updated icons (based on existing pgadmin icon with arrows
>>>>>>>>>> drawn by me)
>>>>>>>>>> 2) add entry to menu as well
>>>>>>>>>> 3) disable icons when not in a transaction
>>>>>>>>>> 4) Ctrl+Shift C|R shortcut
>>>>>>>>>>
>>>>>>>>>> Hopefully all issues are resolved now!
>>>>>>>>>>
>>>>>>>>>> Regards,
>>>>>>>>>>
>>>>>>>>>> John
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> Sent via pgadmin-hackers mailing list (
>>>>>>>>>> pgadmin-hackers(at)postgresql(dot)org)
>>>>>>>>>> To make changes to your subscription:
>>>>>>>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Nikolai Zhubr | 2015-09-26 19:32:24 | Re: Stability and compatability issues on windows xp |
Previous Message | Ashesh Vashi | 2015-09-25 10:20:40 | pgAdmin III commit: Add Menu items, and Toolbar icons for Commit and Ro |