Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Carpool command #4

Open
justinxreese opened this issue Oct 24, 2016 · 9 comments
Open

Carpool command #4

justinxreese opened this issue Oct 24, 2016 · 9 comments

Comments

@justinxreese
Copy link
Member

Should say "dont forget about the #carpool channel if anyone needs a ride to the event tonight" 3 hours before every event

@glowcone
Copy link

Sounds good, I'd like to work on this if it's not assigned yet.

@justinxreese
Copy link
Member Author

@recurza go for it :) I'm having trouble assigning you right now

You'll want the EVENTS_URL environment variable. It is http://cs-jaromir.herokuapp.com/api/v1/events.json

@glowcone
Copy link

glowcone commented Oct 26, 2016

Should I set EVENTS_URL="http://cs-jaromir.herokuapp.com/api/v1/events.json" in process.env or in /bin/hubot?

@glowcone
Copy link

I'm thinking of using node-cron or hubot-cron to do the scheduling, is that a good idea?

@justinxreese
Copy link
Member Author

I'd even consider that a pleasant artifact to reuse elsewhere
On Wed, Oct 26, 2016 at 1:43 AM Justin Reese [email protected]
wrote:

hubot-cron sounds very promising
On Wed, Oct 26, 2016 at 1:31 AM Steph Ng [email protected] wrote:

I'm thinking of using node-cron or hubot-cron to do the scheduling, is
that a good idea?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJNNyg5kuJkl7udT7pgVZnHV_Z8MGDiks5q3uXHgaJpZM4KfNeF
.

@justinxreese
Copy link
Member Author

hubot-cron sounds very promising
On Wed, Oct 26, 2016 at 1:31 AM Steph Ng [email protected] wrote:

I'm thinking of using node-cron or hubot-cron to do the scheduling, is
that a good idea?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJNNyg5kuJkl7udT7pgVZnHV_Z8MGDiks5q3uXHgaJpZM4KfNeF
.

@justinxreese
Copy link
Member Author

The app will know to read that from your environment. You can do 'export
EVENTS_URL=that bin/hubot' as the command if you'd like. Sorry for poor
formatting. Answering from email on my phone. Let me know if that is
unclear
On Tue, Oct 25, 2016 at 10:23 PM Steph Ng [email protected] wrote:

Should I set EVENTS_URL="
http://cs-jaromir.herokuapp.com/api/v1/events.json" in process.env?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJNN3LuiBbvayfdGuqSoyEtNw1Inqn5ks5q3rmigaJpZM4KfNeF
.

@glowcone
Copy link

Got the env variable working! 😁 Also, I found a new module hubot-schedule which seems more suitable for scheduling events.
Now there's a problem: supplybot doesn't know when the events are updated, because it only reads from the json file, so it can't schedule a new timer for the event. Should we tell it to check every time someone does supplybot-list, or only allow events to be scheduled through supplybot?

@justinxreese
Copy link
Member Author

Lets check whenever someone does a list. Thats a good balance.
On Thu, Oct 27, 2016 at 10:51 AM Steph Ng [email protected] wrote:

Got the env variable working! 😁 Also, I found a new module hubot-schedule
which seems more suitable for scheduling events.
Now there's a problem: supplybot doesn't know when the events are updated,
because it only reads from the json file, so it can't schedule a new timer
for the event. Should we tell it to check every time someone does
supplybot-list, or only allow events to be scheduled through supplybot?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJNNyno1VivwVHhF8kdU0f997BNSIAoks5q4LpbgaJpZM4KfNeF
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants