Skip to content

Conversation

@AshesOfOwls
Copy link
Contributor

This is a PR which should depend upon muxinc/media-elements#150 merging.

If there are any updates to the regex in that PR then this should also be updated.


TikTok supports two versions of URL's:

https://www.tiktok.com/@_luwes/video/7527476667770522893
and
https://www.tiktok.com/share/video/7527476667770522893

The second format redirects to the first format, but is still accessible as an embed src. This is useful since TikTok users may change usernames.

@Ritik-gh
Copy link

Would be better to support tiktok player urls as well
sample url - https://www.tiktok.com/player/v1/7526963337926593798

@AshesOfOwls
Copy link
Contributor Author

Would be better to support tiktok player urls as well sample url - https://www.tiktok.com/player/v1/7526963337926593798

@Ritik-gh It seems that the iframe embed does not support that URL from what I have found. For what its worth, it might be easier to transform that locally to /share/video/:video_id . If you have an example of it working I can update both PR's

@Ritik-gh
Copy link

Ritik-gh commented Aug 1, 2025

@AshesOfOwls , it works, you can check this - https://developers.tiktok.com/doc/embed-player
the tiktok video attached on this page is added via an iframe and follows the pattern that I shared above.

@AshesOfOwls
Copy link
Contributor Author

@Ritik-gh Perfect, thanks! I will go back and see what else was wrong to make it fail :)

@Ritik-gh
Copy link

Ritik-gh commented Aug 1, 2025

Cool

@AshesOfOwls
Copy link
Contributor Author

@Ritik-gh Updated, thanks for the suggestion.

@Ritik-gh
Copy link

Ritik-gh commented Aug 3, 2025

Awesome!

Copy link
Collaborator

@luwes luwes left a comment

Choose a reason for hiding this comment

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

👍

@luwes luwes merged commit 7018095 into cookpete:master Aug 12, 2025
5 of 6 checks passed
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.

3 participants