Add force param - Githubissues

ChrisRAoW / mautic-rss-to-email-bundle

Mautic plugin to send emails from RSS

103 stars 23 forks source link

Add force param #40

Open kuzmany opened 3 years ago

kuzmany commented 3 years ago

This just add force param to

{feeditems count="30" force="1"}

This force display items in order in RSS (by default use date)

ChrisRAoW commented 3 years ago

@kuzmany Thanks for your PR. Personally I think "force" a bit misleading / vague. I think an attribute "order" is better with the the options:

rss
date

What do you think?

kuzmany commented 2 years ago

@ChrisRAoW I understand you, but we already use it in production, then It make BC for use. I think it's great improvement and bugfix for some users. It's up to you decide to close it or merge it.

ChrisRAoW commented 2 years ago

@kuzmany What do you mean with "BC for use"?

ChrisRAoW commented 2 years ago

@kuzmany Ah "backwards compatible"....I understand. Will give it another look.