twitter embed issue

[Ritik1330]

twitter embed issue with new url
not work => https://x.com/cb_doge/status/1817235892916146413/photo/1
work =>https://twitter.com/cb_doge/status/1817235892916146413/photo/1

[neSpecc]

Hey. Fill free to fix a regex and we'll release it

[imanaspaul]

@neSpecc I would like to work on it

[neSpecc]

@neSpecc I would like to work on it

@imanaspaul great

[orbachar]

also
https://x.com/MrShibolet/status/1990842843481387410 - works
https://x.com/MrShibolet/status/1990842843481387410?s=20 - doesnt work

[orbachar]

Here’s a quick fix for anyone looking

 embed: {
            class: Embed,
            inlineToolbar: true,
            config: {
                services: {
                    twitter: {
                        regex: /^https?:\/\/(www\.)?(?:twitter\.com|x\.com)\/.+\/status\/(\d+)(?:\?.*)?$/,
                        embedUrl: 'https://platform.twitter.com/embed/Tweet.html?id=<%= remote_id %>',
                        html: '<iframe width="600" height="600" style="margin: 0 auto;" frameborder="0" scrolling="no" allowtransparency="true"></iframe>',
                        height: 300,
                        width: 600,
                        id: ids => ids[1],
                    },
                },
            },
        },

[neSpecc]

@orbachar Would you have time to deliver that regex improvement to the repository?

[orbachar]

of course, will create a PR