search

ciderpunx / twitrssme

TwitRSS.me: Tool to make Twittter timelines and searches into RSS feeds
https://twitrss.me
GNU General Public License v2.0
358 stars 58 forks source link

Correctly parse tweets with multiple images #53

Open TheoLeCalvar opened 6 years ago

TheoLeCalvar commented 6 years ago

48

Quick fix to handle tweets with multiple images.

ciderpunx commented 6 years ago

Thanks for this. Nice one. I'm going to need to test if it has any big performance cost before rolling it out.

TheoLeCalvar commented 6 years ago

Ok. I also have another patch for gifs if you are interested. It is not complete though, only gifs video are supported, others need request to the API to get the url.

rootless4real commented 6 years ago

Can this code handle emojis properly too?

TheoLeCalvar commented 6 years ago

With this new commit it should strip the image and only keep the emoji in the alt attribute. It looked OK but I don't have time to properly test it now.