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

Reset Twitter counts on new landscapes #369

Closed
dankohn opened this issue Sep 29, 2019 · 6 comments
Closed

Reset Twitter counts on new landscapes #369

dankohn opened this issue Sep 29, 2019 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@dankohn
Copy link
Contributor

dankohn commented Sep 29, 2019

Please run y reset-tweet-count and then confirm that the regular nightly update picks up the current count (if any).

@dankohn dankohn added the enhancement New feature or request label Sep 29, 2019
@dankohn
Copy link
Contributor Author

dankohn commented Sep 30, 2019

After removing entry, it did not create a new entry on the next build. AcademySoftwareFoundation/aswf-landscape@cc7b453#diff-ea4f83e4d8108e8fdb17ec90794f9bdf

ASWF currently should show 1:https://twitter.com/search?f=tweets&vertical=default&q=landscape.aswf.io&src=typd

@jordinl
Copy link
Contributor

jordinl commented Sep 30, 2019

@dankohn that's because the Standard Twitter Search API can only search tweets from the last 7 days: https://developer.twitter.com/en/docs/tweets/search/overview. Which means if we reset the tweet count on a landscape and there are tweets that are older than a week, those tweets won't be accounted for. That's why I was suggesting to use the Premium plan, which has access to the full history and can actually perform a count.

@dankohn
Copy link
Contributor Author

dankohn commented Sep 30, 2019

And, I said I don't want to pay. Can you please suggest a way that I can manually insert in the number of tweets to date and then have it count using the standard API going forward? That's what we did for CNCF.

@jordinl
Copy link
Contributor

jordinl commented Sep 30, 2019

@dankohn I could change npm run reset-tweet-count to accept one argument which would be the updated tweet count, so in this case you would execute npm run reset-tweet-count 1. If no count was passed, I would set it to 0. How does that sound?

@dankohn
Copy link
Contributor Author

dankohn commented Sep 30, 2019

Great!

@dankohn
Copy link
Contributor Author

dankohn commented Sep 30, 2019

And once we make that change, could you please manually run it on each of the landscapes (other than CNCF and AI, which are correct).

@jordinl jordinl self-assigned this Sep 30, 2019
jordinl added a commit that referenced this issue Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants