MMM-News

A Former User

@mrdenmark
0_1574001354168_d39a5dca-ac3b-4958-a1b8-a4775e9e74cd-image.png
use bbc-news as source.

dankerthrone

Hey, I’m getting this error message:

[NEWS] Error :  https://newsapi.org/v2/top-headlines?pageSize=20&apiKey=XXX result.code:Required parameters are missing. Please set any of the following parameters and try again: sources, q, language, country, category.

although I think have that configured correctly:

{
			module: "MMM-News",
			position: "bottom_right",
			config: {
					apiKey : "XXX",
					type: "vertical",
					query : [
						  {
							language: "de"
						  }
						  ],
					items: 20,
					timeFormat: "relative",
					endpoint :  "https://newsapi.org/v2/top-headlines",
					scanInterval: 1000*60*10, // This will be automatically recalculated by number of queries to avoid query quota limit. This could be minimum interval.
					templateFile: "template.html"
					
					}
		},

Any help is greatly appreciated :)

A Former User

@dankerthrone
change language to country

uros76

Hi. Is it possible to have two different queries of news, each displayed in a separate horizontal position?

cowboysdude

@uros76 sure it is… see picture below…

Just using the stock newsfeed in MM…[I DETEST CNN but needed a quick example]
You can then use them in the same region or put one each in different regions.
The example is both using bottom_bar but I did try and put one in bottom_bar the other in middle_center and still works… ;)

[Just remember IF you copy the below code and put it in your config.js it will give you an error… you have to change the quotes “” or ‘’ because they do NOT copy correctly].

{
			module: 'newsfeed',
			position: 'bottom_bar',
			config: {
				feeds: [
					{
						title: "CNN",
						url: "http://rss.cnn.com/rss/cnn_latest.rss"
					}
				],
				showSourceTitle: true,
				showPublishDate: true
			}
		},
		{
			module: 'newsfeed',
			position: 'bottom_bar',
			config: {
				feeds: [
					{
						title: "New York Times",
						url: "http://www.nytimes.com/services/xml/rss/nyt/HomePage.xml"
					}
				],
				showSourceTitle: true,
				showPublishDate: true
			}
		},

uros76

Thanks @cowboysdude for the tips. I was more thinking of having two instances of MMM-News module because I like its ability to show a small pic. But multiple uses of MMM-News doesn’t seem to work :/.

Instead, I combined two modules and now have two different feeds working, the second one just without a picture.

cowboysdude

@uros76 said in MMM-News:

MMM-News

Oh I see. I don’t have an api key for that so I can’t really test that one ;) I know you’re setup is not what you wanted but not really sure why running 2 of them is not working without being able to test it.

Here’s an idea… setup up 2 of them again and when you run your mirror run it in ‘npm start dev’ mode and see if you get any errors in the console.

A Former User

not so easy.

MM legacy structure doesn’t consider multiple instances of node_helper. One module should have just one node_helper and vice versa. So, the cloning instance of the module needs some manual fixing. Some simple modules could be clonable easily(like the original newsfeed module), but not this module’s case.

Another point is, to get news individually through cloned modules(If you’ve done somehow), it will consume system resources twice. and API quota also will be consumed doubly. Hmmm. it is not so beautiful.

A reasonable approach would be, Making a serving-slave process(which access the news API) by spawn/fork from one node_helper of the module. then other cloned modules would connect to the slave to get their own needs.

Or make another front-end module, which would just pull data from the original module.

Anyway, I didn’t make this module for that purpose. And under the current MM structure, to do so is not so easy.

Finally, I’m not supporting this module anymore. So, sorry. not possible by me.

uros76

Ok @Sean understand the drawbacks of MM structure and thanks for all the work on this module.
It was just a thought if its possible. I will live with use of two modules :)

sdetweil

@Sean I added support in my MyCovid19 module for multiple instances… it was different, but not hard

on all modlename.js sendSocketNotificatons, add the module identifier
on all node_helper sendSocketNotifications, send it back

in modulename.js , CHECK all receiveSocketNotifcation messages to see if its intended for this instance, as the sendSocketNotification from the node_helper goes to ALL modulename.js instances at once.

then in node_helper I cached the data (as u proposed with the server instance), and for every request fordata from the modulename.js, I check to see if the data has arrived,

if not, the 1st instance to arrive calls the api to get the data, all others block in Promise
once the data arrives, all get resolved() with the data
if the data is already here, then I resolve() with the data

in my production system I have 11 charts drawn, and have 2 extra browser windows open to the MM server, so 33 charts in all, served by one node_helper all sharing one data source, and only making 1 api call for the data

A Former User

@sdetweil
I don’t want to make an argument about your success. The difficulty is always relative. :)

As you’ve said, all cloned modules should manage their own ID by themselves. (not only for the one device but also for each clients if multi-devices are working under server-clients)
It could not be difficult, frankly, usual web sites/applications/components are developed like that.
But the MM… hmmm. I always consider MM would be a standalone running device. Even though there exist some server/client usecases, most of the modules are developed as standalone. Using H/W peripheral(like GPIO or mic) is an example.
Of course, we, developers can make the module clonable from the early stage of development. But I often have a question it’s worthy.

For an imaginary use-case of other users, should I consider to make module clonable? Always my answer for this MM was… no.
Because;

I’m not using clones. If I need something similar, I would make a new module, which might be able to manage multi-views(like CALEXT2) or to share some resources.(AMK2 and HOTWORDS), but it makes things too complex. I hate this way.
I feel some cul-de-sac from MM legacy. MM needs to provide more friendly way to reuse modules. At this moment, not. So that’s why I haven’t considered cloning. and why I lost interest in MM.

Anyway, to say shortly. This module is not made for that purpose. Unless modification by myself or somebody, not possible. That’s all.

sdetweil

@Sean yes, I understand. I have made a couple this way , because the use case shows multiple instances very quickly.

but many do not, if they are hardware constrained…

swvalenti

git pull and npm install says it updated packages but when I check the version in the package.json it shows 2.0.0? What am I doing wrong? Thanks

pi@raspberrypi:~ $ cd MagicMirror 
pi@raspberrypi:~/MagicMirror $ cd modules pi@raspberrypi:~/MagicMirror/modules $ cd MMM-News pi@raspberrypi:~/MagicMirror/modules/MMM-News $ git pull Updating 081fcb2..bff080d 
pi@raspberrypi:~/MagicMirror/modules/MMM-News $ npm install audited 95 packages in 3.8s found 0 vulnerabilities>\

swvalenti

Anyone having an issue of news feed working then not working…just a black screen in the mirror portion position.

A Former User

Yes, i have this problem this week, don’t know really why.
i have not read log for debugging it

Next time, if i have this problem i will see what happen

swvalenti

@Bugsounet ok thanks, let me know if I can do anything to help.

swvalenti

Anything I can do in the mean time to troubleshoot to help get this resolved. It is happening now, I looked in the dev window but didn’t see any errors. I don’t know where else to look. Attached should show the news on the bottom bar but nothing there. Thanks, Scott

swvalenti

@Bugsounet Just circling back on this…would it be easier if I created an issue on github or leave it here?

sdetweil

@swvalenti whos news? did u look in the console window where u started MM?

or pm2 logs if using pm2?

news and calendar use the node_helper to fetch info…

swvalenti

Thanks Sam, checked the pm2 logs and unbeknownst to me…they now went to a paid tier api it seems!


1|MagicMir | [2020-11-17 16:23:16.523] [LOG]    [NEWS] Error :  https://newsapi.org/v2/top-headlines?country=us&pageSize=20&apiKey=99d1bd76c7f140f58c237d54c41ab334 result.code:You have made too many requests recently. Developer accounts are limited to 100 requests over a 24 hour period (50 requests available every 12 hours). Please upgrade to a paid plan if you need more requests.