You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Kudos on creating a project with maintainable code and great documentation! There's a lot being done right here. 👍
I have a small contribution I would like to make: I would like to add a "Keep" filter that will search the Content, not just the Title. I'm willing to submit a pull request for this, and I'm writing here to give a heads up and gather feedback.
I want to try doing this with minimal changes. The simplest way I see to go about this would be to add a checkbox in the UI for "Search Title" that would go below the Block Rules and Keep Rules. This will have minimal disruption, allowing users to opt in to the new feature.
The CSS selector and parsing the fully loaded page are a little overkill for my use case, but I'm hoping that a simple checkbox here will expand the usefulness without adding much complexity.
On the code side it seems that the Entry model already supports the Content field. And the isAllowedEntry function would just need to be updated to take into consideration the new checkbox and the Content field. The value of the checkbox would need to be stored and retrieved, and the feature would need to be documented. New test(s) will be written.
Does anyone know what the status of the Content field is at the time when this function is run?
Is Content populated at this point?
Does Content just contain the RSS contents, not the fully loaded page contents?
Is there anything else I'm missing?
Welcoming any feedback or suggestions.
The text was updated successfully, but these errors were encountered:
So, while Content wouldn't be loaded as a result of scraping when these filters run, I believe it should be loaded due to the code in rss/adapter.go grabbing it directly from the description field in the RSS feed. I assume this is the adapter that would be run for an RSS feed like Webflow's Discourse.
Still investigating why my code doesn't appear to be searching the Content.
Kudos on creating a project with maintainable code and great documentation! There's a lot being done right here. 👍
I have a small contribution I would like to make: I would like to add a "Keep" filter that will search the Content, not just the Title. I'm willing to submit a pull request for this, and I'm writing here to give a heads up and gather feedback.
I want to try doing this with minimal changes. The simplest way I see to go about this would be to add a checkbox in the UI for "Search Title" that would go below the Block Rules and Keep Rules. This will have minimal disruption, allowing users to opt in to the new feature.
I see several open issues with similar requests:
Feature request: Block Rules for article body
Allow block rules based on website content (wants to potentially use a CSS selector to look at a particular part of the page)
Block rules based off of other elements (not just title). (wants to potentially look at the fully loaded page)
The CSS selector and parsing the fully loaded page are a little overkill for my use case, but I'm hoping that a simple checkbox here will expand the usefulness without adding much complexity.
On the code side it seems that the Entry model already supports the Content field. And the isAllowedEntry function would just need to be updated to take into consideration the new checkbox and the Content field. The value of the checkbox would need to be stored and retrieved, and the feature would need to be documented. New test(s) will be written.
Welcoming any feedback or suggestions.
The text was updated successfully, but these errors were encountered: