"skipFirstLoad: true" Doesn't Work?

[JimLynchCodes]

Hi, I am using this code:

const RssFeedEmitter = require('rss-feed-emitter');
const feeder = new RssFeedEmitter({ skipFirstLoad: true });

feeder.add({
    url: 'http://www.nintendolife.com/feeds/news',
    refresh: 2000
});


feeder.on('new-item', function (item) {
    console.log(item);
})

console.log('Feed watcher bot is listening... ')

but it returns a whole mountain of posts. I would think it shouldn't return anything but the console.log statement since the skipFirstLoad: true was passed in... 🤔

[JimLynchCodes]

using node v12.16.1 and rss-feed-emitter v3.1.0

[JimLynchCodes]

same result with node v11.15.0... 😢

[TobiTenno]

our unit tests seem to think it works fine

[midzer]

Can confirm this issue

[djholt]

I can confirm this also.

[mfreiwald]

rss-feed-emitter/src/FeedManager.js

Line 70 in 367cec2

this.populateNewItemsInFeed(data);

Must be changed to this.populateNewItemsInFeed(data, firstLoad);

[filipedeschamps]

🎉 This issue has been resolved in version 3.1.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀