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

[Rodrigo Moya <rodrigo ximian com>]

On Fri, 2004-05-14 at 16:43 +0530, Nicel KM wrote:
> 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.
> 
well, we are using it in the case of 'Send message' type alarms, which
are disabled in the GUI for local backends, but not for connector, IIRC.

cheers