From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Dinesh Kumar <dinesh(dot)kumar(at)enterprisedb(dot)com> |
Cc: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: pgAdmin Event Trigger Compatibility |
Date: | 2013-07-22 13:16:57 |
Message-ID: | CA+OCxow3m2GHtfRLK4L4A=EpEZj+hEtEddbjgz_OXnih29Zg9A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Akshay, please let me know when your review is complete and then I'll take
a look.
On Mon, Jul 22, 2013 at 2:09 PM, Dinesh Kumar <dinesh(dot)kumar(at)enterprisedb(dot)com
> wrote:
> Hi Akshay,
>
> Thank you so much for correcting me on this. It's my bad to forgot the
> naming convention in the code. Now, ::OnOK() is get populating the required
> SQL as per the changes.
>
> Please find the new patch and let me know your valuable inputs.
>
>
> 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>
>
>
> On Mon, Jul 22, 2013 at 5:58 PM, Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> 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*
>>
>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dhiraj Chawla | 2013-07-22 13:17:06 | Re: Search Object dialog Help button issue |
Previous Message | Dave Page | 2013-07-22 13:15:42 | Re: Fixed some minor issues in frmImport |