Re: [PATCH] Add Commit/Rollback toolbar action version 2 - Mailing list pgadmin-hackers

From John Obaterspok
Subject Re: [PATCH] Add Commit/Rollback toolbar action version 2
Date
Msg-id CAOscVdKfDJJYM1nQeXQ=6NO5ySyz7E4750gNe+-GbfX2nT_YEQ@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Add Commit/Rollback toolbar action version 2  (Sanket Mehta <sanket.mehta@enterprisedb.com>)
Responses Re: [PATCH] Add Commit/Rollback toolbar action version 2  (John Obaterspok <john.obaterspok@gmail.com>)
List pgadmin-hackers
Hi Sanket,

Attached patch is rebased now.

Regards,

-- john
          

2015-09-16 11:20 GMT+02:00 Sanket Mehta <sanket.mehta@enterprisedb.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.mehta@enterprisedb.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.vashi@enterprisedb.com> wrote:
Sanket,

Can you please take a look at it?

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


http://www.linkedin.com/in/asheshvashi


On Mon, Sep 14, 2015 at 1:07 AM, John Obaterspok <john.obaterspok@gmail.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@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers





Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: Patch: New field in frmMain statusbar
Next
From: Adam Scott
Date:
Subject: Re: Patch: New field in frmMain statusbar