mkopinsky / zmanimbot

Automatically exported from code.google.com/p/zmanimbot
0 stars 0 forks source link

Status update should update twitter #17

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The admin command to update facebook should be able to update the twitter
status as well.

Original issue reported on code.google.com by zmanim...@gmail.com on 7 Jul 2009 at 2:29

GoogleCodeExporter commented 9 years ago

Original comment by liron.ko...@gmail.com on 7 Jul 2009 at 5:16

GoogleCodeExporter commented 9 years ago
Confirm patch and I will commit code adding twitter status updates!

Original comment by liron.ko...@gmail.com on 8 Jul 2009 at 2:45

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for figuring that out! Moving the bot loading to InitializeBots() is a 
good
idea, I think, though maybe we should call it LoadBots()?

Adding the finally is definitely a good idea!

Why don't we keep the object model we already have, and implement Twitter as 
another
Bot? I'm think that as much as possible, we should keep implementation-specific 
stuff
out of ZmanimBot.java

Original comment by mkopinsky on 8 Jul 2009 at 3:24

GoogleCodeExporter commented 9 years ago
Now?

Original comment by liron.ko...@gmail.com on 9 Jul 2009 at 2:24

Attachments:

GoogleCodeExporter commented 9 years ago
Facebook is handled by facebook app.

Original comment by liron.ko...@gmail.com on 9 Jul 2009 at 3:03

GoogleCodeExporter commented 9 years ago
Fixed in r6

Original comment by liron.ko...@gmail.com on 9 Jul 2009 at 3:46

GoogleCodeExporter commented 9 years ago
Fixed properly in r9

Original comment by liron.ko...@gmail.com on 9 Jul 2009 at 6:02

GoogleCodeExporter commented 9 years ago
Make that r10

Original comment by liron.ko...@gmail.com on 9 Jul 2009 at 6:30