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

Babel: Support tweet URL #9916

Merged
merged 1 commit into from Feb 14, 2021
Merged

Conversation

MrPetovan
Copy link
Collaborator

Depends on friendica/friendica-addons#1080

Highly experimental feature, absolutely no error catching, here be dragons.

@annando
Copy link
Collaborator

annando commented Feb 8, 2021

Tested or untested?

@MrPetovan
Copy link
Collaborator Author

Tested with a successful case:

  • A Twitter tweet URL
  • The tweet exists
  • The twitter addon is correctly set up

It was pivotal in solving #9915, and I may need it in the future, but since it's a debug tool I didn't want to spend that long making sure all the negative cases are gracefully handled.

@annando
Copy link
Collaborator

annando commented Feb 8, 2021

I will have a look at the open PRs this evening. At the weekend I had been busy watching the FOSDEM and needed some time to recreate.

P.S.: Is there any merge order?

@MrPetovan
Copy link
Collaborator Author

Nope, all are independent unless explicitly written (like this one depending on friendica/friendica-addons#1080)

@annando annando merged commit 6321dcc into friendica:develop Feb 14, 2021
@MrPetovan MrPetovan deleted the task/babel-twitter-url branch February 14, 2021 23:31
@MrPetovan
Copy link
Collaborator Author

Please don't forget about friendica/friendica-addons#1080 this PR depends on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants