MMM-News and BBC

DarrenHill

https://newsapi.org/v2/top-headlines?sources=bbc-news&pageSize=100&apiKey=YOUR_KEY

I get this, which looks slightly different to yours? :

{"status":"ok","totalResults":10,"articles":[{"title":"Vaping: Juul to pay $438.5 m in youth marketing probe","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T19:52:21.045172Z","url":"http://www.bbc.co.uk/news/business-62813896"},{"title":"Eliza Fletcher: Police identify body found in search for heiress","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T16:52:18.6995928Z","url":"http://www.bbc.co.uk/news/world-us-canada-62811661"},{"title":"Ukraine war: Russians to pay €45 more to enter EU under new rules","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T16:37:23.3671715Z","url":"http://www.bbc.co.uk/news/world-europe-62811675"},{"title":"Fat Leonard: Malaysian businessman linked to US Navy scandal escapes","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T16:22:22.2671956Z","url":"http://www.bbc.co.uk/news/world-us-canada-62764226"},{"title":"Canada stabbings: Police hunt remaining suspect after Saskatchewan attacks","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T15:22:24.1190163Z","url":"http://www.bbc.co.uk/news/live/world-us-canada-62811629"},{"title":"Ukraine: Ben Stiller and Sean Penn latest Americans banned from Russia","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T12:52:14.8462267Z","url":"http://www.bbc.co.uk/news/world-europe-62806655"},{"title":"Canada stabbings: Relatives identify 'random' victims of Saskatchewan attack","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T11:07:21.1055569Z","url":"http://www.bbc.co.uk/news/world-us-canada-62804826"},{"title":"Liz Truss: Boris Johnson to give farewell speech before new PM takes office","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-06T04:37:21.3806544Z","url":"http://www.bbc.co.uk/news/live/uk-politics-62802807"},{"title":"Keith Davis: He was protecting the oceans - then he disappeared","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-05T20:52:23.0241991Z","url":"http://www.bbc.co.uk/news/world-us-canada-62603911"},{"title":"Timed Teaser: How did Putin mark Gorbachev's death?","author":"BBC News","source":{"Id":"bbc-news","Name":"BBC News"},"publishedAt":"2022-09-05T20:37:23.1798468Z","url":"http://www.bbc.co.uk/news/world-us-canada-62763889"}]}

@sdetweil I’ll do your test now…

MMRIZE

@DarrenHill
Ok, I found something. I don’t know why, but your result has “articles.source.Name” instead of “articles.source.name” that makes this error.

sdetweil

@MMRIZE yes, was just about to post that

DarrenHill

@sdetweil I get this (for the relevant bit):


[06.09.2022 21:11.46.552] [LOG]   fetch result= {
  status: 'ok',
  totalResults: 10,
  articles: [
    {
      title: 'Vaping: Juul to pay $438.5 m in youth marketing probe',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T19:52:21.045172Z',
      url: 'http://www.bbc.co.uk/news/business-62813896'
    },
    {
      title: 'Eliza Fletcher: Police identify body found in search for heiress',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T16:52:18.6995928Z',
      url: 'http://www.bbc.co.uk/news/world-us-canada-62811661'
    },
    {
      title: 'Ukraine war: Russians to pay €45 more to enter EU under new rules',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T16:37:23.3671715Z',
      url: 'http://www.bbc.co.uk/news/world-europe-62811675'
    },
    {
      title: 'Fat Leonard: Malaysian businessman linked to US Navy scandal escapes',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T16:22:22.2671956Z',
      url: 'http://www.bbc.co.uk/news/world-us-canada-62764226'
    },
    {
      title: 'Canada stabbings: Police hunt remaining suspect after Saskatchewan attacks',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T15:22:24.1190163Z',
      url: 'http://www.bbc.co.uk/news/live/world-us-canada-62811629'
    },
    {
      title: 'Ukraine: Ben Stiller and Sean Penn latest Americans banned from Russia',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T12:52:14.8462267Z',
      url: 'http://www.bbc.co.uk/news/world-europe-62806655'
    },
    {
      title: "Canada stabbings: Relatives identify 'random' victims of Saskatchewan attack",
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T11:07:21.1055569Z',
      url: 'http://www.bbc.co.uk/news/world-us-canada-62804826'
    },
    {
      title: 'Liz Truss: Boris Johnson to give farewell speech before new PM takes office',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-06T04:37:21.3806544Z',
      url: 'http://www.bbc.co.uk/news/live/uk-politics-62802807'
    },
    {
      title: 'Keith Davis: He was protecting the oceans - then he disappeared',
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-05T20:52:23.0241991Z',
      url: 'http://www.bbc.co.uk/news/world-us-canada-62603911'
    },
    {
      title: "Timed Teaser: How did Putin mark Gorbachev's death?",
      author: 'BBC News',
      source: [Object],
      publishedAt: '2022-09-05T20:37:23.1798468Z',
      url: 'http://www.bbc.co.uk/news/world-us-canada-62763889'
    }
  ]
}
[06.09.2022 21:11.46.635] [LOG]   [NEWS] Error :  https://newsapi.org/v2/top-headlines?sources=bbc-news&pageSize=100&apiKey=<redacted> TypeError: Cannot read properties of undefined (reading 'toString')
    at slugify (/home/pi/MagicMirror/modules/MMM-News/node_helper.js:24:17)
    at cb (/home/pi/MagicMirror/modules/MMM-News/node_helper.js:89:33)
    at getArticles (/home/pi/MagicMirror/modules/MMM-News/node_helper.js:136:22)
    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)

sdetweil

@DarrenHill said in MMM-News and BBC:

  source: [Object],

its in there

DarrenHill

@MMRIZE said in MMM-News and BBC:

@DarrenHill
Ok, I found something. I don’t know why, but your result has “articles.source.Name” instead of “articles.source.name” that makes this error.

Anything I can do to overcome that?

No idea why I’m getting different API returns to you guys…?

sdetweil

@DarrenHill @MMRIZE

also doesn’t get ‘content’ or ‘description’, so those will fail too…

DarrenHill

I tried to register for a new api key, but when I tried it told me that they don’t support the UK (where I am) so I wonder if that somehow has something to do with it.

This has obviously changed since I first registered, and given I’m getting some response from the API it is at least partially working, but besides that I have no idea…

mumblebaj

@DarrenHill Hi. Apologies for the late reply. I am on a different timezone than the other guys.

I would suggest using MMM-NewsAPI. The same API key that you are using with MMM-News will work.

DarrenHill

@mumblebaj - Thanks, no need to apologise.

I tried the other module, and it also doesn’t show anything besides it’s “News” header title.

The config I tried was:

        {
            module: "MMM-NewsAPI",
            header: "News",
            position: "bottom_center",
            config: {
                apiKey : "<my API key>",
                domains: "bbc.co.uk",
                language: "en"
                }
        },

And also I tried replacing the domains: with sources: “bbc-news” and that gave the same results.

There’s nothing in the logs, aside from confirming that the module is loading (as can be seen from the displayed title too).

sdetweil

@DarrenHill from the doc

Add DEBUG Handling.
Only add debug: true to the config section if you are experiencing issues and require more information to find issues.

sdetweil

@DarrenHill domins or sources must be inside the much more complex query section of the config

see the doc

                    query: {
                            country: "",
                            category: "",
                            q: "",
                            qInTitle: "",
                            sources: "",
                            domains: "cnn.com,nytimes.com,news24.com",
                            excludeDomains: "",
                            language: ""
                    }

mumblebaj

@DarrenHill MMM-NewsAPI has a whole lot more configs. See below sample.

{
                        module: "MMM-NewsAPI",
                        header: "news",
                       //  disabled: true,
                        position: "bottom_bar",
                        pages: {"sport": "bottom_bar"},
                        config: {
                                apiKey: <api-key>,
                                type: "horizontal",
                                choice: "everything",
                                pageSize: 20,
                                sortBy: "publishedAt",
                                drawInterval: 1000*30,
                                templateFile: "template.html",
                                fetchInterval: 1000*60*60,
                                query: {
                                        country: "",
                                        category: "",
                                        q: "",
                                        qInTitle: "",
                                        sources: "",
                                        domains: "bbc.co.uk",
                                        excludeDomains: "",
                                        language: ""
                                        }
                                }
				},

DarrenHill

@mumblebaj - Thanks for the support, and for both modules.

Unfortunately that gives me a headline in Spanish (I think) about Apple iPhone 14 with “undefined” below it.

I also tried your everything example configs from your GH page and got a similar layout (headline and undefined).

I’m beginning to think NewsAPI itself is screwed for me unfortunately, so we may be onto a loser here regardless of module.

sdetweil

@mumblebaj i commented out your disabled: true,

mumblebaj

@DarrenHill That is very weird as the configs seem to work for me and the other guys who have tried to assist thus far.

DarrenHill

@mumblebaj - Yes, as I said I’m starting to think that your modules are both working fine, but the API feed I’m getting from NewsAPI is somehow corrupted and they are universally having issues with it.

I can’t create a new API key as it won’t let me (it says they don’t support UK now if you try and create a new key from here), and I guess that might be related somehow?

So I think we might be at a dead end here unfortunately with anything that’s using that API as its source.

mumblebaj

@DarrenHill The data coming through is fine as it works for me with the source that you provided. Not sure why you not able to create an API key though. I just tested creating another API Key and that worked as well. Unfortunately I am not able to assist with getting a key created.

sdetweil

@DarrenHill see my chat message

sdetweil

@mumblebaj newsapi.org free is only 100 calls per day

I guess 4/hour is ok