Skip to content
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

Item source is Feedzy when using multiple feeds #899

Closed
AndreeaCristinaRadacina opened this issue Mar 4, 2024 · 2 comments · Fixed by #903
Closed

Item source is Feedzy when using multiple feeds #899

AndreeaCristinaRadacina opened this issue Mar 4, 2024 · 2 comments · Fixed by #903
Assignees
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. medium (1d) - This label is used for issues that can be completed within 1 day or less. Priority-Medium Expected resolution time - up to 1 month.

Comments

@AndreeaCristinaRadacina
Copy link

Description

While working on a ticket, I found that when you're using multiple feed sources in the same import, the source will be "Feedzy" instead of the actual name of the feed.

Not sure if it's a bug or if it's expected.

ref: https://secure.helpscout.net/conversation/2525118325/406402?viewId=212385

Step-by-step reproduction instructions

  1. Go to Feedzy > Imports
  2. Create a new import
  3. Use the following feeds: https://community.511tactical.com/feed/, https://loadoutroom.com/feed/
  4. In the Map Content area, use Item Full Content and Item Source
  5. Save and check one of the posts

Screenshots, screen recording, code snippet or Help Scout ticket

When using only one feed:
image

When using both:
image

Environment info

No response

Is the issue you are reporting a regression

No

@AndreeaCristinaRadacina AndreeaCristinaRadacina added the bug This label could be used to identify issues that are caused by a defect in the product. label Mar 4, 2024
@pirate-bot pirate-bot added the customer report Indicates the request came from a customer. label Mar 4, 2024
@vytisbulkevicius vytisbulkevicius added the Priority-Medium Expected resolution time - up to 1 month. label Mar 4, 2024
@Soare-Robert-Daniel Soare-Robert-Daniel added the medium (1d) - This label is used for issues that can be completed within 1 day or less. label Mar 7, 2024
@Soare-Robert-Daniel
Copy link
Contributor

After taking a deep look at this, the full content fetch is done using an endpoint from our server to fetch all the data from the desired feeds.

The endpoint gives all the collected feeds into a single XML. When there are multiple sources, the item_source is set to Feedzy since it is the name of the pages that contain all feeds.

With a single source:

item_feed_title: Celebrating Women in Service: International Women’s Day at 5.11 Tactical
tem_feed_link: https://www.511tactical.com/community/celebrating-women-in-service/
item_source_title: 5.11 Community
item_source_link: https://www.511tactical.com/community/

With multiple sources:

item_feed_title: Celebrating Women in Service: International Women’s Day at 5.11 Tactical
item_feed_link: https://www.511tactical.com/community/celebrating-women-in-service/
item_source_title: Feedzy
item_source_link: https://feedzy.themeisle.com/wp/wp/index.php/feed/feedzyfpv3?token=0837331df9

To solve this, I think changing the endpoint from https://github.com/Codeinwp/feedzy-full-post will be required to append the source origin of the feed. But I am not sure what major changes it will be for the parsing mechanism of the endpoint plugin to extract those.

@vytisbulkevicius
Copy link
Contributor

Thanks for looking into it @Soare-Robert-Daniel.

Not a big priority, but I would like to have this fixed, I remember some requests about this from our customers in the past as well, so unless it's too complicated to make this change, let's do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. medium (1d) - This label is used for issues that can be completed within 1 day or less. Priority-Medium Expected resolution time - up to 1 month.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants