Add periodic triggers - Githubissues

instedd / mbuilder

Scalable SMS application builder

http://mbuilder.instedd.org/

6 stars 1 forks source link

Add periodic triggers #30

Closed waj closed 9 years ago

waj commented 9 years ago

When clicking on "new trigger" a submenu should be presented with an additional option: "periodic task". Show the existing type of trigger as: "Incoming message".

The "periodic task" trigger header, should provide 4 options: "daily", "weekly", "monthly" and "yearly". Each option should present the configuration shown in the screenshots attached.

The rest of the trigger should behave identically to the incoming message one.

Imported from BitBucket: https://bitbucket.org/instedd/mbuilder/issue/30
Originally Reported By: Nicolas di Tada
Originally Created At: 2013-09-10 18:01:27+00:00

waj commented 9 years ago

The "periodic task" trigger doesn't have a "from number" nor "message" fields, right? You just have access to the tables.

What trigger do you need to do right now? Do you need filtering a table by a hardcoded value? (because there's no way to do that right now).

Original comment by Ary Borenszweig on 2013-09-10 18:01:27+00:00

waj commented 9 years ago

For now we only need to send a static message to all phone numbers in a table. In a while we'll add filtering by harcoded values.

Original comment by Nicolas di Tada on 2013-09-10 18:01:27+00:00

waj commented 9 years ago

in which time of the day should we run this periodic triggers? Should we let the user set a desired hour for the trigger to start?

Original comment by Mariano Coca on 2013-09-10 18:01:27+00:00

waj commented 9 years ago

Yes, sorry, missed that with re-using the Things UI. We should include a time of the day field and run the cron every hour or so.

Original comment by Nicolas di Tada on 2013-09-10 18:01:27+00:00