MMM-Pnews

DeanoAus

Hey mate, I think I have a bug for you. :bug:

In MMM-PNews.js you hard code in the sorting method (latest) for the source.

this.url = "https://newsapi.org/v1/articles?source=" + this.config.newsSource + "&sortBy=Latest&apiKey=";

Latest isn’t a valid sorting method for some sources.
Example is the https://newsapi.org/abc-news-au-api

If you hit that with the url they provide with Latest in there for sorting, it fails and gives

{"status":"error","code":"sourceUnavailableSortedBy","message":"The news source you've selected (abc-news-au) isn't available sorted by latest"}

But if you change Latest to Top in the url, works fine. :)

This is causing some feeds to not work.

DeanoAus

This post is deleted!

DeanoAus

Also, mine doesn’t seem to be updating the feed. (Sorry!)

Seeing this in the log?
fatal: unable to access ‘https://github.com/cowboysdude/MMM-PNews/’: GnuTLS recv error (-54): Error in the pull function.

cowboysdude

@DeanoAus said in MMM-Pnews:

sourceUnavailable

I’ve updated the url so that should fix that…

The other error is a git error … “: Error in the pull function”

DeanoAus

Awesome, thanks! :D

lucallmon

@cowboysdude Just installed this for the first time. Looks GREAT! However, it seems to take up a lot of space at the bottom bar; I am using a somewhat smaller screen. Is there a way I can limit the module to more of the center leaving space on both sides at the bottom for smaller modules? Also, I’d like to center the pic and text. It seems that it only aligns to the left. Any help would be greatly appreciated.

cowboysdude

@lucallmon I’ll take a look at it later when I get home :) We’ll see what we can do!

Johans

hi Cowboysdude,

I like your modules, nice work.
I was wondering if you could modify or make a separate module for the dutch (belgian) newsfeeds.
take a look at this side: http://feeds.nieuwsblad.be/nieuws/snelnieuws
they provide for each headline, a pircture and short description.
It would be nice if you could also incorparate this into the module.

Johan

cowboysdude

@Johans said in MMM-Pnews:

hi Cowboysdude,

I like your modules, nice work.
I was wondering if you could modify or make a separate module for the dutch (belgian) newsfeeds.
take a look at this side: http://feeds.nieuwsblad.be/nieuws/snelnieuws
they provide for each headline, a pircture and short description.
It would be nice if you could also incorparate this into the module.

Johan

Johan, Thank you! I will certainly take a peek at that this evening!! :)

cowboysdude

@Johans

Here you go:

https://github.com/cowboysdude/MMM-DNews

Mykle1

@cowboysdude said in MMM-Pnews:

Here you go:

So, all I have to do is ask and you’ll make me a module?

You’re awesome! :^)

cowboysdude

@Mykle1 said in MMM-Pnews:

@cowboysdude said in MMM-Pnews:

Here you go:

So, all I have to do is ask and you’ll make me a module?

You’re awesome! :^)

Sure, why not? LOL

Johans

@cowboysdude

You are amazing.
Never thought it would be realized.
Had a hard day on the job today, but this makes my day.
I’m going to install now.
Thanks

Johan

goprojojo

@cowboysdude said in MMM-Pnews:

@Johans

Here you go:

https://github.com/cowboysdude/MMM-DNews

Isnt Johan looking for Dutch instead of Danish news? :P

Johans

@cowboysdude

just installed, but got an error -

Load script: modules/MMM-Dnews//MMM-Dnews.js
MMM-Dnews.js Failed to load resource: the server responded with a status of 404 (Not Found)
loader.js:190 Error on loading script: modules/MMM-Dnews//MMM-Dnews.js
script.onerror @ loader.js:190

Would like to make a printscreen, but i can’t manage to do that on the raspberry.

(indeed Dutch news, not Danish - thanks goprojojo )
Edit: added screenprints

cowboysdude

Sorry about that… right now I just don’t have time. I am SUPER busy… I will look at it when I can.

cowboysdude

@Johans said in MMM-Pnews:

MMM-Dnews.js

The config was wrong…

Should be:

       module: 'MMM-DNews',
      position: 'bottom_bar',
      config: {
        rotateInterval: 20 * 1000
		    }
               },

cowboysdude

@goprojojo Yes I believe you are right!! OPPS…I’m fixing that right now!

@Johans

There NOW if your config.js is ‘nl’ then the translation file should work now :)

The other issue was my fault because on github I put ‘MMM-Dnews’… should be ‘MMM-DNews’… all fixed!

do a git pull to update :) Should work now.

PS you may have to adjust translation file… I used google translate .

cowboysdude

@Johans said in MMM-Pnews:

@cowboysdude

just installed, but got an error -

Load script: modules/MMM-Dnews//MMM-Dnews.js
MMM-Dnews.js Failed to load resource: the server responded with a status of 404 (Not Found)
loader.js:190 Error on loading script: modules/MMM-Dnews//MMM-Dnews.js
script.onerror @ loader.js:190

Would like to make a printscreen, but i can’t manage to do that on the raspberry.

(indeed Dutch news, not Danish - thanks goprojojo )
Edit: added screenprints

As for the weather error…it does that every now and then and it’s the weather api …it fixes itself so no worry there :)

Johans

@cowboysdude
works like a charm now.
Thank you very much for this and all the great work you are doing.
much appriciated !!

Johans