jasonmayes / Twitter-Post-Fetcher

Fetch your twitter posts without using the new Twitter 1.1 API. Pure JavaScript! By Jason Mayes
http://www.jasonmayes.com/projects/twitterApi
Other
910 stars 234 forks source link

Can you take a look #179

Closed webguy262 closed 6 years ago

webguy262 commented 6 years ago

I wrote here the other day about TwitterFetcher not working here.

I posted about then issue on SO and get this reply

Can you advise on a fix that will get TwitterFetcher working?

jasonmayes commented 6 years ago

In the HTML you import Twitter Fetcher and after that you then try and reference it right? Update: just viewed your source - it appears so. That is very odd. Let me have a look. Something must be conflicting with it, however if you go to the js console and type twitterFetcher it finds it after the page load so it does get there eventually....

jasonmayes commented 6 years ago

What happens if you put the call to twitter Fetcher in the same file as Twitter Fetcher itself at the end?

var config1 = { "profile": {"screenName": 'bigmouthinc_'}, "domId": 'example1', "maxTweets": 2, "enableLinks": true, "showUser": false, "showInteraction": false }; twitterFetcher.fetch(config1);

jasonmayes commented 6 years ago

Oh wait as the guy said it appears you have AMD on your site which is deferring the loading of twitterFetcher. Yes I think if you remove the AMD stuff it should be fine.

jasonmayes commented 6 years ago

Here is TwitterFetcher without the extra checks: https://jsfiddle.net/jfqhmv0b/

webguy262 commented 6 years ago

Same error, just points to the call in twitterFetcher_min: ReferenceError: twitterFetcher is not defined http://www.bigmouthinc.com/wp-content/themes/bmi/js/twitterFetcher_min.js:77 It's on the site that way now if you want to look. Thanks for you help!

On Wednesday, June 6, 2018, 4:28:37 PM EDT, Jason Mayes <notifications@github.com> wrote:  

What happens if you put the call to twitter Fetcher in the same file as Twitter Fetcher itself?

var config1 = { "profile": {"screenName": 'bigmouthinc_'}, "domId": 'example1', "maxTweets": 2, "enableLinks": true, "showUser": false, "showInteraction": false }; twitterFetcher.fetch(config1);

— You are receiving this because you authored the thread.

Reply to this email directly, view it on GitHub, or mute the thread.:

|

Can you take a look · Issue #179 · jasonmayes/Twitter-Post-Fetcher

I wrote here the other day about TwitterFetcher not working here. I posted about then issue on SO and get this r... |

|

|

|

Build software better, together

GitHub is where people build software. More than 28 million people use GitHub to discover, fork, and contribute ... |

|

|

ReferenceError: twitterFetcher is not defined http://www.bigmouthinc.com/wp-content/themes/bmi/js/twitterFetcher_min.js:77

jasonmayes commented 6 years ago

You got it working using my modified script yes? See above.

webguy262 commented 6 years ago

Yes, the modified script fixed the issue. Thank you very much!