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

Adds the possibility to hide link previews #28

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

Conversation

gabearro
Copy link

Adds an additional parameter to the sendTweet method + the internal request one. This will allow users to toggle whether or not they want link previews.

@gabearro
Copy link
Author

gabearro commented Dec 24, 2024

Realized this isn't complete with respect to the CLI. To be honest, I do not think it's necessary to add this to the CLI.

@Freytes Freytes closed this Dec 29, 2024
@gabearro
Copy link
Author

@Freytes how come you closed this PR? It adds a relatively useful functionality

@Freytes Freytes reopened this Dec 30, 2024
@Freytes
Copy link
Collaborator

Freytes commented Dec 30, 2024

@gabearro it was closed in error, please resolve your conflicts and I will pull it in.

@gabearro
Copy link
Author

@Freytes no worries! I was worried I didn't comply with some contribution policy I didn't know about. I've fixed the conflicts and checked that it builds + works correctly.

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