Skip to content
This repository has been archived by the owner on Feb 12, 2019. It is now read-only.

Adding Simple HTML5 Caching (every minute) #303

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

briankerrdesign
Copy link

For those running into Rate Limiting - https://dev.twitter.com/docs/rate-limiting

Kinda tested....

For those running into Rate Limiting - https://dev.twitter.com/docs/rate-limiting

Kinda tested....
@purcell
Copy link
Collaborator

purcell commented Nov 8, 2012

Interesting. If I'm understanding your intention and code correctly, it looks like this would not support multiple widgets on one page, which isn't ideal... The cache key should probably be related to the API request URL.

@briankerrdesign
Copy link
Author

Yes, I stored the data under a single key in the local array. You could
make that ket specific to each the widget with an ID or hash.]

**
Brian Kerr
Designer & Developer

[email protected]
www.briankerrdesign.com

On Thu, Nov 8, 2012 at 12:53 PM, Steve Purcell [email protected]:

Interesting. If I'm understanding your intention and code correctly, it
looks like this would not support multiple widgets on one page, which isn't
ideal... The cache key should probably be related to the API request URL.


Reply to this email directly or view it on GitHubhttps://github.com//pull/303#issuecomment-10204456.

@purcell
Copy link
Collaborator

purcell commented Nov 8, 2012

Cool. I'll have a play with this code when I get some time. Thanks for sending it over! :-)

@mattstauffer
Copy link

Hm, I took a look at this and it sort of looks like there's an infinite chain of caching. load() caches it into tweetData, and then passes the cached information into extractData, which passes the same information back into tweetData if it's an odd minute. Am I missing something, though?

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

Successfully merging this pull request may close these issues.

3 participants