-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
feat(route): add route for BBC Twitter accounts #15275
Open
danilo-delbusso
wants to merge
1
commit into
DIYgod:master
Choose a base branch
from
danilo-delbusso:route/bbc-user
base: master
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+63
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BBC Twitter accounts may contain feeds different from those available on bbc.co.uk or similar. For instance, the Breaking News feed on Twitter contains fewer posts and is more relevant for just major news. This route uses Twitter Feed fetched from BBC articles, and then uses the existing BBC route to fetch the articles and add their content on the `description`, which avoids having to open the URLs. It also makes use of the `twitter/user` route as a base route to fetch the tweets in the first place.
Successfully generated as following: http://localhost:1200/twitter/bbc-user/BBCBreaking - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCWorld - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCNews - Failed ❌
|
github-actions
bot
added
the
Auto: Route Test Complete
Auto route test has finished on given PR
label
Apr 17, 2024
Successfully generated as following: http://localhost:1200/twitter/bbc-user/BBCBreaking - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCWorld - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCNews - Failed ❌
|
Successfully generated as following: http://localhost:1200/twitter/bbc-user/BBCBreaking - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCWorld - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCNews - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCNewsAsia - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCAfrica - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCBusiness - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCLondonNews - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCNorthAmerica - Failed ❌
http://localhost:1200/twitter/bbc-user/BBCSport - Failed ❌
|
Locally seems to work just fine - I assume all the 503 errors here are related to Twitter API / request limitations? Someone more knowledgeable might know |
This PR is stale because it has been opened for more than 3 weeks with no activity. Comment or this will be closed in 7 days. |
keep alive please |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Involved Issue / 该 PR 相关 Issue
N/A
Example for the Proposed Route(s) / 路由地址示例
New RSS Route Checklist / 新 RSS 路由检查表
Puppeteer
Note / 说明
Reason for adding route
BBC Twitter accounts may contain feeds different from those available on bbc.co.uk or similar.
For instance, the Breaking News feed on Twitter contains fewer posts and is more relevant for just major news.
Based on existing routes
This route uses Twitter Feed fetched from matching BBC accounts, and then uses the existing BBC route to fetch the articles and add their content on the
description
, which avoids having to open the URLs. It also makes use of thetwitter/user
route as a base route to fetch the tweets in the first place.Note this only works on BBC Twitter accounts that posts BBC Articles, such as twitter.com/BBCBreaking, twitter.com/BBCNews, twitter.com/BBCWorld, etc. More can be seen here: https://twitter.com/search?q=BBC&f=user