Skip to content

Conversation

@n7studios
Copy link
Contributor

Summary

Removes emojis from the WordPress Post's Permalink when importing Broadcasts that contain emojis in the title.

Whilst emojis are supported in Permalinks, it's better to exclude them for readability, link sharing and SEO.

WordPress' sanitize_title will remove non-alphanumeric characters, and replace spaces with hyphens - however won't cover removal of emojis.

Testing

  • testGeneratePermalinkFunction: Tests that emojis are removed from the given Broadcast title when parsed through the generate_permalink function.

Checklist

@n7studios n7studios self-assigned this May 19, 2025
@github-actions
Copy link

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios changed the base branch from main to use-wp-doing-cron-method May 19, 2025 06:01
@github-actions
Copy link

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios requested review from a team, corydhmiller and noelherrick and removed request for a team May 19, 2025 08:21
@n7studios n7studios marked this pull request as ready for review May 19, 2025 08:21
@n7studios n7studios merged commit 14d9b67 into use-wp-doing-cron-method May 20, 2025
77 checks passed
@n7studios n7studios deleted the broadcasts-exclude-emojis-from-permalinks branch July 12, 2025 03:03
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.

3 participants