Re: pgAdmin Event Trigger Compatibility

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Dinesh Kumar <dinesh(dot)kumar(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin Event Trigger Compatibility
Date: 2013-07-22 12:28:29
Message-ID: CANxoLDcvJyBbAn-5FKGO9RaHAekkMOKpHAf9_bTW+wkH75RoQA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dinesh

I have just started reviewing your code and found on issue where any
changes made on Event Trigger dialog won't be saved. After looking at the
code, you haven't override the ::OnOK() function in dlgEventTrigger.cpp.
Can you please fix this and resend the new patch.

On Wed, Jul 10, 2013 at 9:46 PM, Dinesh Kumar <dinesh(dot)kumar(at)enterprisedb(dot)com
> wrote:

>
> Yes, we can change this enable check box as a radio button. But,
>>> "REPLICA/ALWAYS" are two enable's properties. Hence, We have implemented
>>> this in the proposed way. Kindly share your opinion on this.
>>>
>>
>> So: "Enabled Replica ( ) Enabled Always ( ) Disabled ( )" ?
>>
>
> Yes Dave.
>
>
>
> Dinesh
>
> --
> *Dinesh Kumar*
> Software Engineer
>
> Ph: +918087463317
> Skype ID: dinesh.kumar432
> www.enterprisedb.co <http://www.enterprisedb.com/>m<http://www.enterprisedb.com/>
> *
> Follow us on Twitter*
> @EnterpriseDB
>
> Visit EnterpriseDB for tutorials, webinars, whitepapers<http://www.enterprisedb.com/resources-community> and
> more <http://www.enterprisedb.com/resources-community>
>

--
*Akshay Joshi
Senior Software Engineer
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
Phone: +91 20-3058-9522
Mobile: +91 976-788-8246*

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dhiraj Chawla 2013-07-22 12:57:37 Search Object dialog Help button issue
Previous Message Akshay Joshi 2013-07-22 09:31:56 Fixed some minor issues in frmImport