-
Notifications
You must be signed in to change notification settings - Fork 177
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
Slug: allow unicode characters #12603
Conversation
Plugin builds for 257cb2d are ready 🛎️!
|
Size Change: +1.82 kB (0%) Total Size: 2.72 MB
ℹ️ View Unchanged
|
it('should allow using unicode characters', () => { | ||
const result = cleanForSlug('హల-వరలడ'); | ||
expect(result).toBe('హల-వరలడ'); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets add a data set of different slugs and expected return. Maybe test Cyrillic script, Arabic, Other unicode characters, accented characters like È, hindi characters and mandarin chinese.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a few more in 7565ca8. Accented characters are already handled in a previous test (see the first test of the file).
Note: the encoded slug shows up properly in the address bar within Chrome and within the story editor link display Verified with Kanji characters as well https://stories-qa-wordpress-amp.pantheonsite.io/web-stories/kanji-%e6%bc%a2%e5%ad%97-2/ |
Context
Summary
Allows unicode characters in the slug. Also, adds using
remove-accents
package.Relevant Technical Choices
To-do
User-facing changes
Testing Instructions
This PR can be tested by following these steps:
Reviews
Does this PR have a security-related impact?
Does this PR change what data or activity we track or use?
Does this PR have a legal-related impact?
Checklist
Type: XYZ
label to the PRFixes #12546