Skip to content
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

Add a note to README #50

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

SamTheGeek
Copy link

Adding a note to the section of the readme about fetching new tweets, noting that your account must be publicly visible since the bearer token can only see public tweets. This caused me some heartache when trying to use this functionality

Adding a note to the section of the readme about fetching new tweets, noting that your account must be publicly visible since the bearer token can only see public tweets.
@nhoizey
Copy link
Contributor

nhoizey commented Dec 19, 2022

@SamTheGeek would you have a link to Twitter's documentation where we can read that the bearer token only allows getting public tweets?

@zachleat
Copy link
Contributor

I would also add a note about the limitation that images cannot be fetched (via eleventy-img) from protected accounts!

@SamTheGeek
Copy link
Author

@nhoizey It's on the Oauth 2.0 page, under bearer token.

This method is typically for developers that need read-only access to public information.

Elsewhere on the developer site, the app-only methods are described as

App only Access Token allows a Twitter developer app to access information publicly available on Twitter.

@nhoizey
Copy link
Contributor

nhoizey commented Jan 12, 2023

@SamTheGeek could you please add the link to https://developer.twitter.com/en/docs/authentication/oauth-2-0 in your PR, and the additional mention of images as suggested by @zachleat, so I can merge?

@nhoizey nhoizey added the documentation Improvements or additions to documentation label Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants