MMM-RottenTomatoes displays awaiting update

haywire

Sorry to hear it didn’t work for you. I am making the assumption your using the original pull which were from https://github.com/AdamMoses-GitHub/NPMJS-rt-scraper.
Unfortunately, I modified the original MMM-RottenTomatoes, so I don’t have a like for a like environment, but assumed correction would work for the original module.

The root of the problem is Rotten Tomato new web format, which is part of rt-scraper (which you found).

As indicate my fix were based on another user correction. You can attempt to pull or replace using his build.

https://github.com/parnic/NPMJS-rt-scraper

He has also updated his repo for RottenTomatoes module it self and updated the dependency. I haven’t tried it, but seem he best overall re-package solution.
https://github.com/parnic/MMM-RottenTomatoes

If that still doesn’t work and have time I can re-package my update, but it would be base on slight modified version of MMM-RottenTomatoes.

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:

Go to the node-modules/rt-scraper directory in your MagicMirror installation dir.
Edit index.js
Replace the three instances of “homepage-top-box-office” with “movies_sidebar”
Replace the request uri rottentomatoes.com to rottentomatoes.com/m/dream_horse
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.