Skip to content

feat: add support for 'link' field in conditions#1135

Merged
Soare-Robert-Daniel merged 2 commits into
developmentfrom
feat/url_filtering
Aug 13, 2025
Merged

feat: add support for 'link' field in conditions#1135
Soare-Robert-Daniel merged 2 commits into
developmentfrom
feat/url_filtering

Conversation

@RaduCristianPopescu
Copy link
Copy Markdown
Contributor

Summary

Add URL-based filtering option to exclude specific domains

Will affect visual aspect of the product

NO

Screenshots

CleanShot 2025-08-11 at 17 06 40@2x

Closes https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/836

@RaduCristianPopescu RaduCristianPopescu self-assigned this Aug 11, 2025
@RaduCristianPopescu RaduCristianPopescu added the pr-checklist-skip Allow this Pull Request to skip checklist. label Aug 11, 2025
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Aug 11, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for a new 'link' field in RSS feed filtering conditions, allowing users to filter RSS items based on their URL to exclude specific domains.

  • Adds 'link' as a new filterable field in the conditions system
  • Implements link value extraction from RSS feed items
  • Configures appropriate operator restrictions for the link field

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
js/Conditions/ConditionsControl.js Adds 'link' field configuration to the supported fields array with operator restrictions
includes/util/feedzy-rss-feeds-conditions.php Implements link value extraction using get_link() method in the evaluation logic

Comment thread includes/util/feedzy-rss-feeds-conditions.php
@pirate-bot
Copy link
Copy Markdown
Contributor

pirate-bot commented Aug 11, 2025

Plugin build for f75a56e is ready 🛎️!

Note

You can preview the changes in the Playground

@poonam279
Copy link
Copy Markdown

@Soare-Robert-Daniel, does the link filter apply only to the domain URL, and not to the full post URL/link?

@Soare-Robert-Daniel
Copy link
Copy Markdown
Contributor

@Soare-Robert-Daniel, does the link filter apply only to the domain URL, and not to the full post URL/link?

It applies to the URL in its format given by the feed. For this feed https://www.nasa.gov/feeds/iotd-feed/ the filter will apply to elements in the <link> tag like <link>https://www.nasa.gov/image-detail/hubble-30dor-potw2531a-2/</link>.

In this case, it will be the full link.

@Soare-Robert-Daniel Soare-Robert-Daniel merged commit c41e76b into development Aug 13, 2025
9 checks passed
@Soare-Robert-Daniel Soare-Robert-Daniel deleted the feat/url_filtering branch August 13, 2025 12:13
@pirate-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist. released Indicate that an issue has been resolved and released in a particular version of the product.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants