MMM-RottenTomatoes displays awaiting update

[ifzer0c00lsucked]

@haywire it worked. took some time for the first update but after that it has been working perfectly. THANKS!

[tomyboy96]

Hey @haywire!

I tried this but it did not work for me…

Tried removing old MMM-RottenTomatoes and installing https://github.com/parnic/MMM-RottenTomatoes

It got the “Awaiting update” message fixed but the only thing showing up now is the:
box office, opening this week, coming soon
without any movies.

Using the same config.js as I did for the old MMM-RottenTomatoes

Also tried manually removing rt-scraper and then cloning from https://github.com/parnic/NPMJS-rt-scraper but still no success…

Any ideas?

[airs747]

I found a (dirty) workaround.
Inspecting https://www.rottentomatoes.com/m/dream_horse, you can see the sidebar that used to be on the homepage. The structure is the same as before, except that the name of the div-element is different than assumed in the rt-scraper code.

So, this is what I did to get it working:

1. Go to the node-modules/rt-scraper directory in your MagicMirror installation dir.
2. Edit index.js
3. Replace the three instances of “homepage-top-box-office” with “movies_sidebar”
4. Replace the request uri rottentomatoes.com to rottentomatoes.com/m/dream_horse
5. Restart MagicMirror and you should now see the RT movie listing appearing again.

Now, this is dirty, because of the Dream Horse link. I’m think about a better way (take the first movie on the site?) and commit to github. But for now we can at least enjoy the movie listings again.

[sdetweil]

@airs747 said in MMM-RottenTomatoes displays awaiting update:

’m think about a better way (take the first movie on the site?) and commit to github

find the github repo for the module you fixed,
fork it (now YOU have a linked ‘copy’)

make the same changes, commit to YOUR copy
and then offer a Pull Request (new pull request button) to the source to fix the problem for all

[4r3st3r]

Hi, thanks for all the help so far! When I cloned the parnic’s rt-scraper and MMM-RottenTomatoes fork it still just displayed “awaiting update”. Has anyone else managed to get this module fixed?

[rdemets]

@airs747 Hi. I tried that, but I didn’t find any instance of homepage-top-box-office in index.js. I can see in the code source the movies_sidebar, but I don’t know where to place it. Can you please give more details about what you did ? I’ve installed RottenTomatoes via parnic code, and the rt-scrapper linked to it. Thank you very much.

[rdemets]

@tomyboy96 Same here, did you get it to work ? Thanks

[tomyboy96]

@rdemets
Nope… still nothing…

[lavolp3]

@sdetweil @airs747
There is already a fix that is PRd to the rt-scraper module. Unfortunately since February already.
The problem is that it would need to be merged and published as an npm module to be properly installed here.

[tomyboy96]

@lavolp3

Is this something that can be installed manually?