MMM-News and BBC

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: ""
                    }