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

Resolves issue #78 #96

Closed
wants to merge 1 commit into from
Closed

Resolves issue #78 #96

wants to merge 1 commit into from

Conversation

ixti
Copy link

@ixti ixti commented Aug 9, 2011

Improves the way strings are got sluggized in order to resolve #78

@ixti
Copy link
Author

ixti commented Aug 9, 2011

Basically I guess we need clean conventions on what to replace and what to not, because, I guess ' must be simply removed, with this patch, string Don't be a fool. Seriously. will become sluggized as don-t-be-a-fool-seriously, but I think the best slug in this case should be dont-be-a-fool-seriously.

@cloudhead, what do you think?

@ixti
Copy link
Author

ixti commented Aug 9, 2011

Will reopen once all tests are ready and will realize strict ruleset of replacements

@ixti ixti closed this Aug 9, 2011
@ixti ixti reopened this Aug 10, 2011
@ixti
Copy link
Author

ixti commented Aug 10, 2011

Hm. so strange. pull request shows only one commit, but actually there are more than one after last update.

@ixti ixti closed this Aug 11, 2011
@ixti
Copy link
Author

ixti commented Aug 11, 2011

Can't update commit range. So created new pull request #98

@ixti ixti mentioned this pull request Aug 11, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Routing breaks if there is a dot in the name of an article
1 participant