Re: [evolution-patches] Patch for 57564: reminder message option for appointments

[Nicel KM <mnicel novell com>]

On Fri, 2004-05-14 at 13:02 +0200, Rodrigo Moya wrote:
> On Fri, 2004-05-14 at 14:13 +0530, Nicel KM wrote:
> > Hi Rodrigo,
> >     Thanks for your checkin. It requires a similar check in
> > button_options_clicked_cb() so that the options dialog is invoked even
> > if email alarms are not supported by backend. Attached patch adds this
> > check. Please let me know if it's ok.
> > Fixes #57564
> > 
> looks ok, please commit

committed, thanks.

> > PS: also, do we really need the alarm_email_address in this options
> > dialog otherwise?
> >
> I guess we do to allow the user to change it, right?

I guess not. The options that we give are to repeat the alarm and for
entering the message to be displayed.

regards,
-Nicel.