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

apply with twitter ( without views stats ) #271

Merged
merged 1 commit into from
Jun 22, 2023
Merged

Conversation

hichri-louay
Copy link
Contributor

PR Description

Hey team,

This PR aims to address a change in our Twitter API usage. We're transitioning back to Twitter API v1.1 from v2, mainly due to the cost associated with the v2 version.

As a result of this shift, we can gather all required information about a specific link, except for its 'Views'. We're aware of the importance of this data, and we're planning to develop a feature for scraping views in the future.

Changes Made

  • Switched back from Twitter API v2 to v1.1 due to cost considerations.
  • Adjusted our data retrieval mechanisms to function effectively with API v1.1, now able to retrieve all link-related information, with the exception of 'Views'.
  • Laid groundwork for future implementation of a view-scraping feature.

Notes for Reviewers

  • Please review the changes made, focusing particularly on the API transition and its potential effects on the overall functionality.
  • Verify that the system works correctly with Twitter API v1.1, and that all link-related data, barring 'Views', is retrieved successfully.
  • We'd like to draw your attention to the planned view-scraping feature. We welcome any suggestions or recommendations that could aid us in this future implementation.

Thank you for your time and contributions. Your feedback and suggestions are greatly appreciated.

Best regards,
Louay HICHRI

@ksibisamir ksibisamir merged commit 75617ba into develop Jun 22, 2023
3 checks passed
@ksibisamir ksibisamir deleted the apply-twitter-link branch June 22, 2023 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants