Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Probably misunderstanding how feed filters work #1270

Closed
pdxmph opened this issue Feb 29, 2024 · 1 comment
Closed

Probably misunderstanding how feed filters work #1270

pdxmph opened this issue Feb 29, 2024 · 1 comment

Comments

@pdxmph
Copy link

pdxmph commented Feb 29, 2024

Describe the bug
Using the url.contains filter doesn't seem to work, but its inverse does. I am guessing I am missing how this is supposed to work.

To Reproduce
Steps to reproduce the behavior:

  1. Edit a feed
  2. Set up a filter to capture URLs with advice in them (url.contains('advice'))
  3. Refresh the feed.
  4. Nothing is marked as read, including things that have advice in the URL.
  5. Revisit the filter and negate it with not url.contains('advice')`
  6. Refresh. Everything but advice columns is marked read.

Expected behavior
url.contains('advice') will mark anything with advice in the URL as read. Use case: My local paper puts all the advice columns under https://oregonlive.com/advice/foo.

Environment (please complete the following information):

  • CommaFeed version (or "commafeed.com"): 4.3.1 (fafd4c9)
  • Browser [e.g. chrome, firefox]: Firefox
  • Device [e.g. desktop, mobile]: Desktop (Linux, Fedora 39)

Additional context
Add any other context about the problem here.

@Athou
Copy link
Owner

Athou commented Mar 1, 2024

Hi @pdxmph,

3. Refresh the feed.

The filter only applies to new entries, changing the filter and refreshing the feed will not mark existing entries as read.

Refresh. Everything but advice columns is marked read.

I'm not sure how that's possible :D

There's an issue here to apply the filter to existing entries #728

Repository owner locked and limited conversation to collaborators Mar 23, 2024
@Athou Athou converted this issue into discussion #1304 Mar 23, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants