railsyard / railsyardcms

Railsyard - One Ruby on Rails CMS to rule them all!
www.railsyardcms.org
Other
95 stars 7 forks source link

twitter_stream snippet can be instantiated multiple times now #3

Closed spieker closed 13 years ago

spieker commented 13 years ago

Have added the snippet id to the templates now and have "fixed" thi twitter_stream snippet

silviorelli commented 13 years ago

Great work! I'll merge it tomorrow morning as I need to change a little thing on how you managed the snippet class. Time to go bed now :)

silviorelli commented 13 years ago

There is an error with the twitter stream snippet you modified. I get: "NetworkError: 400 Bad Request - http://api.twitter.com/1/statuses/user_timeline.json?screen_name=silviorelli&count=5&include_rts=1&callback=jQuery15106113634815232316_1313746203488&_=1313746203697"

spieker commented 13 years ago

Hi,

I have had this error too. While debuging I had a look at the response of twitter. There was a message, that one client is not allowed to do more then 120 (?) requests per hour/day (I'm not sure now and I'm in the office, so I can't try it now). The twitter snippet is doing an update every 20 seconds, so you have 120 request after having the page open for about 40 minutes.

I think the timeout for loading new tweets should be longer than 20 seconds.

Greetings, Paul

silviorelli commented 13 years ago

Yes I also thought about the too frequent polling, now I'm setting it at 90 sec. I fixed also other bugs with the username.

spieker commented 13 years ago

ok, thanks!