search

aurelg / feedspora

FeedSpora posts RSS/Atom feeds to your social network accounts.
35 stars 5 forks source link

Tweepy client and/or mkrichtext formatting bug #34

Closed wilddeej closed 5 years ago

wilddeej commented 5 years ago

I've noticed a few tweets produced by FeedSpora that have incorrect formatting. An real-life example is:

AD: New! "Introverts Unite" T-Shirt: #introversion | #unity #causes #activism #satire <link>

The hashtags are the problematic detail: this client is configured to only include 4 hashtags, and clearly 5 have been included. Additionally, the title/hashtag inclusion formatting is funky, using both a colon and a pipe as data separators. Although minor, this is incorrect and needs resolution (as well as a pertinent test case to detect/report this issue, obviously).

aurelg commented 5 years ago

Do you still have the source RSS feed?

wilddeej commented 5 years ago

Sort of, although not every item manifests the problem. It should not be difficult to reproduce the issue for a mkrichtext and/or TweepyClient test case, however.

On Thu, Dec 6, 2018 at 12:30 PM Aurélien Grosdidier < notifications@github.com> wrote:

Do you still have the source RSS feed?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/aurelg/feedspora/issues/34#issuecomment-444999635, or mute the thread https://github.com/notifications/unsubscribe-auth/AXIWhpo4hivd2eacUdXrLEOX2muQBqHrks5u2XBEgaJpZM4ZFVb0 .

wilddeej commented 5 years ago

There are now tests in place that have the same sort of feed formatting that caused the noted problem. This issue has definitely been fixed, at least with my latest PR for Issue #40

wilddeej commented 5 years ago

This issue is now fixed as of PR #46