use replaceAll for dehyphenation

[jenul-ferdinand]

This fixes an issue with the dehyphenateUrl function, which previously only replaces the first hyphen in a string. For URLs with multiple hyphens (e.g., foo-bar-foo), would incorrectly return foo bar-foo.

The fix was to use replaceAll() instead of replace().

[Copilot]

Pull request overview

This PR fixes a bug in the dehyphenateUrl function where only the first hyphen was being replaced instead of all hyphens. The fix changes replace() to replaceAll() to correctly handle presenter names with multiple hyphens in URL parameters.

• Updated dehyphenateUrl to use replaceAll() instead of replace() for complete hyphen replacement

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

[Copilot]

The fix for dehyphenateUrl should include test coverage to verify that all hyphens are replaced correctly. Consider adding tests that cover:

• Single hyphen: "foo-bar" → "foo bar"
• Multiple hyphens: "foo-bar-baz" → "foo bar baz"
• No hyphens: "foobar" → "foobar"
• Edge cases: empty string, string with only hyphens, etc.

This would prevent regression of the original bug where only the first hyphen was replaced.