-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[InstagramBridge] Truncate long titles and use full text as content #961
[InstagramBridge] Truncate long titles and use full text as content #961
Conversation
Thanks for the PR!
It makes sense to be consistent for all items in a feed in my opinion.
Does something like this work for you? rss-bridge/bridges/FacebookBridge.php Lines 664 to 665 in 5305c40
Let me know if you want to update the PR, I have no objections merging it like it is. |
I had something different for the truncation on words, but I guess it's ok if already used in the Facebook bridge. Do you want to use a 64 chars limit as in Facebook, or keep my proposed 120?
Sure, I will. |
If 120 works for you, it's fine by me. in the end it depends on how your feed reader displays the titles. Since I don't use Instagram, this is no issue for me :) |
I think the PR is now ready to merge:
|
Merged. Thanks for keeping it updated 👍 |
…SS-Bridge#961) - Truncate long titles and use full text as content (using only the first line of text content as title)
Instagram posts' text can be very long used as feed items' title. This PR truncates titles over 120 characters, and uses the full text as items' content if truncated.
Possible changes I can make on this PR if you prefer: