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

Correct minor typo #65

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

Correct minor typo #65

wants to merge 2 commits into from

Conversation

popey
Copy link

@popey popey commented Jan 4, 2023

I just downloaded my archive and the first line of tweets.js is window.YTD.tweets.part0 = [ not window.YTD.tweet.part0 = [ as stated in the README.md

I just downloaded my archive and the first line of `tweets.js` is `window.YTD.tweets.part0 = [` not `window.YTD.tweet.part0 = [` as stated in the `README.md`
@@ -34,7 +34,7 @@ Take ownership of your Twitter data. First talked about at [Jamstack Conf 2019](
### Populate the database from your Twitter Archive zip

1. Copy `./data/tweets.js` from your [Twitter Archive](https://help.twitter.com/en/managing-your-account/how-to-download-your-twitter-archive) `zip` file into the `./database` directory of this project.
* Rename `window.YTD.tweet.part0` in `tweets.js` to `module.exports`
* Rename `window.YTD.tweets.part0` in `tweets.js` to `module.exports`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@popey could you add " (formerly window.YTD.tweet.part0)", as this is the name it had before?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot that there is already #47

So either change yours and I'll merge the first that is ready, or close this one as duplicate.

@nhoizey nhoizey added documentation Improvements or additions to documentation duplicate This issue or pull request already exists labels 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 duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants