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

[PcGamerBridge] fix pc gamer, revert to meta tags, get image from meta tag as enclosure, get category tags #2271

Merged
merged 6 commits into from
Oct 19, 2021

Conversation

mdemoss
Copy link
Contributor

@mdemoss mdemoss commented Sep 21, 2021

I expect using meta tags and a.article-link should be less likely to break in the future.
If a.article-link should break, then finding each a for the same domain and checking for the meta tags where og:type is article ought to be even more durable.

@em92
Copy link
Contributor

em92 commented Oct 19, 2021

Hi, @mdemoss !
Снимок экрана_2021-10-19_08-31-51
Are type_news and channel_gaming_software supposed to be in feed item categories? IMHO they duplicate Category: News and Category: gaming software

@mdemoss
Copy link
Contributor Author

mdemoss commented Oct 19, 2021

Are type_news and channel_gaming_software supposed to be in feed item categories? IMHO they duplicate Category: News and Category: gaming software

I'm not sure why they have some of those tags, maybe it's for their internal use. I filtered out a couple already at

array('van_buying_guide_progressive', 'serversidehawk')

Maybe a better approach could be to remove every tag that contains an underscore, at the risk of losing some meaningful tags.

@em92 em92 merged commit 8f98e07 into RSS-Bridge:master Oct 19, 2021
floviolleau pushed a commit to floviolleau/rss-bridge that referenced this pull request Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants