-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add modify links action #974
Conversation
@girishpanchal30, thank you for taking care! Here are my findings:
|
Can you please share Feed URL?
It could be a cache issue on your end; everything looks fine from my side. |
@girishpanchal30 here is the feed: https://wpshout.com/feed |
@AndreeaCristinaRadacina I've fixed the mentioned issue with the latest commit. |
@girishpanchal30 I am not sure why, but when using the Modify links action, the import no longer works: Instance:
Also, from the previous comment (just making sure), have you checked this too?
|
@AndreeaCristinaRadacina It seems issue with PHP 8, now it fixed with the latest commit. |
@girishpanchal30, great! It works as expected now. Thank you! |
🎉 This PR is included in version 4.4.14 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
In this PR, I've added modify links action to modify the
<a></a>
HTML attributes.Screenshots
With Pro Version:
https://tinyurl.com/2ypn2944
https://tinyurl.com/23qrd67z
Without Pro version:
https://tinyurl.com/27xpoqg2
https://tinyurl.com/2b3gelle
Check before Pull Request is ready:
Closes https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/647