fix(router/twitter): fix twitter converstaion root post missing issue - #22963
fix(router/twitter): fix twitter converstaion root post missing issue#22963E1xP wants to merge 1 commit into
Conversation
|
Successfully generated as following: http://localhost:1200/twitter/user/KOMODO_Games_JP - Failed ❌ |
|
Auto Review No clear rule violations found in the current diff. Previously reported issues (the self-reply filter change in |
8297d26 to
6916004
Compare
|
Successfully generated as following: http://localhost:1200/twitter/user/KOMODO_Games_JP - Failed ❌ |
|
Had change the code base on the Auto Review. |
|
The route test failure is due to the test enviroment not config the |
|
@TonyRL This PR is for fixing the tweet which send in converstaion post will missing. And also could you review the below PR both are working around the twitter route? feat(route/twitter): change retweet author to original post account- PR #22965 fix(route/twitter): Add X-Client-Transaction-Id header for url UserTweetsAndReplies PR #22967 Thanks for you time. |
Involved Issue / 该 PR 相关 Issue
Close #22938
Example for the Proposed Route(s) / 路由地址示例
New RSS Route Checklist / 新 RSS 路由检查表
PuppeteerNote / 说明
This fix is try to contain the twitter conversation tweet post into the route /twitter/user.
As some twitter acount using the conversation to create post.
But currently the root post is filter due to not access the entries 'profile-conversation-'.
The change is adding 'profile-conversation-' to the getUserTweetsByID and filter the none root post info.
And sorry for the misformat pull request before.