Skip to content

Fix thumbnail PHP type hint#4216

Merged
Alkarex merged 2 commits intoFreshRSS:edgefrom
Alkarex:fix-thumbnail
Feb 9, 2022
Merged

Fix thumbnail PHP type hint#4216
Alkarex merged 2 commits intoFreshRSS:edgefrom
Alkarex:fix-thumbnail

Conversation

@Alkarex
Copy link
Member

@Alkarex Alkarex commented Feb 9, 2022

#fix #4215

@Alkarex Alkarex merged commit 4b9d66f into FreshRSS:edge Feb 9, 2022
@Alkarex Alkarex deleted the fix-thumbnail branch February 9, 2022 22:37
@Frenzie
Copy link
Member

Frenzie commented Feb 9, 2022

What's a feed that has some btw?

@Alkarex
Copy link
Member Author

Alkarex commented Feb 11, 2022

What's a feed that has some btw?

@Frenzie We take the first enclosure as the "thumbnail".

@Frenzie
Copy link
Member

Frenzie commented Feb 11, 2022

Right, what's a sample feed with enclosures? I may have 100+ feeds but none of them use enclosures.

@Alkarex
Copy link
Member Author

Alkarex commented Feb 11, 2022

See for instance http://feeds.feedburner.com/hackaday/LgoM

<media:thumbnail url="[https://hackaday.com/wp-content/uploads/2022/02/twist-feature.png]()" />
<media:content url="[https://hackaday.com/wp-content/uploads/2022/02/twist-feature.png]()" medium="[image]()">
	<media:title type="[html]()">twist-feature</media:title>
</media:content>

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.

Single Items showing up even when more are avaiable in Normal View

2 participants