Autoresponder: Start and end date

Discussion in 'Developers' Forum' started by wyrie, Dec 4, 2009.

?

Should start and end times be added as an option to the autoresponder

Poll closed Dec 11, 2009.
  1. yes

    7 vote(s)
    87.5%
  2. no

    1 vote(s)
    12.5%
  1. wyrie

    wyrie New Member

    Hi All,

    One of the features I miss from other panels is the ability to set start and end times for an autoresponder. At the moment we have to make notes of when to switch off which particular responder as suppose to the autoresponder automatically being activated/de-activated.

    I would like to add this feature. Would anyone else like the same thing? and how do you see that being implemented?

    If implemented should start and end dates replace the enabled tickbox or should backward compatibility be kept and the user can choose.
     
  2. till

    till Super Moderator Staff Member ISPConfig Developer

    I think thats a nice feature. But I would keep the checkbox.
     
  3. Horfic

    Horfic Member

    When it is done, i will update my plugin to make it available for the email user.
     
  4. giftsnake

    giftsnake New Member

    please keep the checkbox and just add the start and end time :)
     
  5. wyrie

    wyrie New Member

    Great, I'll add the dates and keep the current checkbox. Added to bugtracker.
     
  6. wyrie

    wyrie New Member

    Attached is a screenshot of the autoresponder page with date selection added. It's a little different to the other forms but logically it seems better to group them.

    Adding them as rows in the form obfuscates usability (for me at least) and after all the dates are related to whether or not the responder is active.

    Comments?
     

    Attached Files:

  7. till

    till Super Moderator Staff Member ISPConfig Developer

    Looks nice. I think it would be good to make it "active and date" and not "active or date". What do you think?
     
  8. wyrie

    wyrie New Member

    I think that depends on the functionality we want. Personally "active and date" would be my preference.

    But I didn't want to force the user into setting dates if they didn't want to. Maybe the default state of the date fields should be '--'?
     
  9. wyrie

    wyrie New Member

    Hi All,

    I've added the dates feature and tested on debian & gentoo. Hope you like the way it works.

    @Horfic let me know if there is anything I can do RE your plugin.
     
  10. Horfic

    Horfic Member

    Till just needs to modify the remote command and i can rebuild the autoresponder. Otherwise you can send me your working version for the ispconfig and I will change it my self and send it then to tim.
     
  11. till

    till Super Moderator Staff Member ISPConfig Developer

    Hi,

    I think the remote command has not to be changed as all new parameters for a function are available automatically as the remoting API uses the same data definition files then the normal interface.
     
  12. till

    till Super Moderator Staff Member ISPConfig Developer

    I've just read trogh the code. Really good work! Nice to see that you extended the tform class with the datetime type.
     
  13. wyrie

    wyrie New Member

    :) Thanks, glad you like it.
     

Share This Page