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

Tweet Feature #1772

Merged
merged 11 commits into from
Feb 17, 2024
Merged

Tweet Feature #1772

merged 11 commits into from
Feb 17, 2024

Conversation

HanilJain
Copy link
Contributor

Added Feature #1239

using tweepy library this feature has been added.

BEARER_TOKEN
APP_KEY
APP_KEY_SECRET
ACCESS_TOKEN
ACCESS_TOKEN_SECRET
needed to be set in the .env file

here is an example video

tweet_feature.mp4

@HanilJain
Copy link
Contributor Author

@DonnieBLT could you please review

@DonnieBLT
Copy link
Collaborator

Can you please add instructions to get those values?

Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

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

I think we already had sent to Twitter somewhere else in the code. Can you check where that is and see if that's working and maybe updates at that section

@HanilJain
Copy link
Contributor Author

I think we already had sent to Twitter somewhere else in the code. Can you check where that is and see if that's working and maybe updates at that section

yes there was but not was used anywhere.
and dose not seems to contain API v2 and complicated
So I removed it.
is it okay ?

@HanilJain
Copy link
Contributor Author

@DonnieBLT DonnieBLT enabled auto-merge (squash) February 17, 2024 19:27
@DonnieBLT DonnieBLT merged commit 7f1d740 into OWASP-BLT:main Feb 17, 2024
5 checks passed
@HanilJain HanilJain deleted the issue-1239 branch February 17, 2024 19:41
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.

None yet

2 participants