search

cloudhead / toto

the 10 second blog-engine for hackers
MIT License
1.49k stars 244 forks source link

Resolves issue #78 #96

Closed ixti closed 13 years ago

ixti commented 13 years ago

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

ixti commented 13 years ago

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 commented 13 years ago

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

ixti commented 13 years ago

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

ixti commented 13 years ago

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