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.

Sign up for GitHub

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

Jump to bottom

slugify with all caps does not work as expected #464

Closed
paulovieira opened this issue Nov 30, 2015 · 3 comments
Closed

slugify with all caps does not work as expected #464

paulovieira opened this issue Nov 30, 2015 · 3 comments

Comments

@paulovieira
Copy link

I'm not sure if this is an issue or not, but I was surprised with the following behaviour:

s.slugify("ABC XYZ")returns "a-b-c-x-y-z". I think the expected value would be "abc-xyz"
@dperrymorrow
Copy link
Contributor

i have experienced this as well in the past with capitalization.
Is this considered a bug? if so i can attempt to do a pull

@esamattis
Copy link
Owner

Yes this is a bug but I have a feeling we've fixed this already once :) PR welcome if it happens on the master too.

And yeah abc-xy is the expected result.

@paulovieira
Copy link
Author

Yes, I confirm that it is fixed in master (fixed with #448).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dperrymorrow @esamattis @paulovieira