feedzy-rss-feeds icon indicating copy to clipboard operation
feedzy-rss-feeds copied to clipboard

Invalid URLs in Feed Categories

Open stefan-cotitosu opened this issue 1 year ago • 6 comments

Description

I was trying to group three URLs in a feed category, and they are flagged as invalid, although the import job works properly, with all of them entered separately as sources within the same import job.

The problem might be specific to feeds from this website. Below are the used URLs:

https://medium.com/feed/craft-ventures
https://medium.com/feed/@dfkoz
https://medium.com/feed/omers-ventures

Step-by-step reproduction instructions

  1. Create an import job and use all these three source
  2. use the Item Content magic tag and run the import - it works properly
  3. Add those three URLs in a feed category (I tried to add them both as one URL per row, and on the same line with commas and no space in between) -> all of them are marked as invalid.
  4. I test with two sources only, to avoid the once with the @ character in the URL, the same error occurs

Screenshots, screen recording, code snippet or Help Scout ticket

Import job use case

Image

it works: https://feedzy.stefancoti.com/category/medium-com/


Feed categories use case

Image Image


Support ticket: https://secure.helpscout.net/conversation/2776750310

Environment info

https://pastebin.com/3n2CphT1

Is the issue you are reporting a regression

No

stefan-cotitosu avatar Dec 01 '24 13:12 stefan-cotitosu

@stefan-cotitosu, I’ve tested this on my local setup and a TasteWP instance, and everything appears to be working fine on my end.

You can verify it using my test instance:
https://hugeincrease.s4-tastewp.com/wp-admin/post.php?post=6&action=edit
Login details:
Username: girish
Password: girish

Thanks!

girishpanchal30 avatar Dec 03 '24 05:12 girishpanchal30

@girishpanchal30 Thanks for checking. I have checked your dashboard, and there is no error while combining the feeds in category but when I created an import using that feed category. I can see the errors:

Image

However, the simplepie is showing the posts for all three feeds. Can you please check?

poonam279 avatar Dec 09 '24 12:12 poonam279

@poonam279 I've double-checked, and I can't seem to replicate the issue. I'm not sure what I'm missing. Could you please share your test instance?

Thanks!

girishpanchal30 avatar Dec 30 '24 09:12 girishpanchal30

Hey @girishpanchal30, The behavior of these feeds is very inconsistent. Currently, I am unable to replicate the same issue on a new test instance. However, I noticed that when sources are added as one URL per row in a category, they all appear as invalid. In contrast, if I enter them on a single line, separated by a comma and a space, the same URLs start working. I have recorded a screencast for you: https://vertis.d.pr/v/e9a8C6.

Could you please try to replicate the same issue on your end?

poonam279 avatar Jan 02 '25 13:01 poonam279

@poonam279 I've added support for a per-row separator in this PR. Please review it and let me know if you encounter any issues.

You can now add feed URLs as shown in the example below:

https://medium.com/feed/craft-ventures
https://medium.com/feed/@dfkoz
https://medium.com/feed/omers-ventures

Thanks

girishpanchal30 avatar Feb 19 '25 11:02 girishpanchal30

@girishpanchal30, I have tested and it's working with the latest build. Thank you.

poonam279 avatar Feb 24 '25 15:02 poonam279