miniflux / v2

Minimalist and opinionated feed reader
https://miniflux.app
Apache License 2.0
6.54k stars 706 forks source link

Allow multiple categories for a feed #2575

Open lloydjatkinson opened 4 months ago

lloydjatkinson commented 4 months ago

Improving existing features is more important than adding new ones.

I believe this feature meets this requirement.


Hi,

I would like to be able to add multiple categories to a feed. Several feeds I subscribe to have overlapping categories, and it's awkward putting a feed under one category when really it fits two or more. So what I end up doing is either have everything under the existing "All" category, or have super vague and generic categories which somewhat defeats the purpose.

This is where I am drawing inspiration from:

image

What are your thoughts on this? 😄

Sleepful commented 4 months ago

Hi,

I submitted an issue here with this idea too.

I proposed 3 different ways of implementation for the UX, the #2 fits your issue!

Check here:

https://github.com/miniflux/v2/issues/2573

I would be happy to implement any of the three options.

Sleepful commented 4 months ago

Waiting for feedback from @rdelaage tho

rdelaage commented 4 months ago

Don't know why you wait for my feedback :). Personally I thing it is ok, people that do not want multiple categories can just use one category. Reading the miniflux website I found that https://miniflux.app/docs/migration.html. It was possible to use multiple categories for a feed in Miniflux v1 but this was removed in v2 by the author. The author doesn't want a feature that goes against the philosophy of the software (minimalism) so don't know if this is a wanted feature (there is no formal definition of the minimalism).

Sleepful commented 4 months ago

oh sorry @rdelaage , i meant to tag @fguillot , whom I assume is the main maintainer. I must have confused you because of the profile picture. oops :)

Like you said though:

The author doesn't want a feature that goes against the philosophy of the software (minimalism) so don't know if this is a wanted feature (there is no formal definition of the minimalism).

This is why I want to hear approval before submitting a PR.

fguillot commented 4 months ago

This feature has been requested several times in the past if you search through this issue tracker.

I'm not against this feature, but I am concerned about the number of changes required to implement it properly.

Not only does the database schema have to be modified, but it also impacts different parts of the application, such as the Miniflux API, the Fever API, the web UI, etc.