Skip to content
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

[2.x] Support Chinese slug #1907

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

thingasd
Copy link
Contributor

Modify the slugify function to accept Chinese character as slug, it should work with setting appropriate config twill.slug_utf8_languages

@thingasd thingasd changed the title do not replace chinese characters when format permalink Support Chinese slug Nov 14, 2022
@haringsrob
Copy link
Contributor

Holding this one back for testing with #1897

@haringsrob
Copy link
Contributor

Hey @thingasd

Could you tell me what is expect to be the slug for when I for example enter:

  • test-知道
  • 知道
  • 知道bar知道

Kind regards,

@thingasd
Copy link
Contributor Author

Hey @thingasd

Could you tell me what is expect to be the slug for when I for example enter:

  • test-知道
  • 知道
  • 知道bar知道

Kind regards,

test-知道
知道
知道bar知道

@haringsrob
Copy link
Contributor

ok, so slugs can actually contain chinese characters?

@thingasd
Copy link
Contributor Author

ok, so slugs can actually contain chinese characters?

This pr allows slugs to contain Chinese characters, which can be used in the URL for improving the readability. i.e. http://www.example.com/例子

@ifox ifox changed the title Support Chinese slug [2.x] Support Chinese slug Mar 29, 2023
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