From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | John Obaterspok <john(dot)obaterspok(at)gmail(dot)com> |
Cc: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>, Ashesh Vashi <ashesh(dot)vashi(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 07:58:19 |
Message-ID: | CA+OCxoy6H_MVA5wbaLbLgyLico0K-vhJvVLjozD4MWV-AVcOhQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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 | Ashesh Vashi | 2015-09-25 08:00:44 | Re: [PATCH] Add Commit/Rollback toolbar action version 2 |
Previous Message | John Obaterspok | 2015-09-25 06:03:24 | Re: [PATCH] Add Commit/Rollback toolbar action version 2 |