Navigation Menu

Skip to content
This repository has been archived by the owner on Mar 5, 2021. It is now read-only.

Person slugs include title PK, not text #95

Closed
evildmp opened this issue May 9, 2013 · 1 comment
Closed

Person slugs include title PK, not text #95

evildmp opened this issue May 9, 2013 · 1 comment

Comments

@evildmp
Copy link
Owner

evildmp commented May 9, 2013

For example:

5-daniele-procida

instead of

mr-daniele-procida

Likely to have been introduced in #68 #70

@darbula
Copy link
Contributor

darbula commented May 27, 2013

This should be fixed on this line.

Documentation states prepopulated_fields doesn’t accept DateTimeField, ForeignKey, nor ManyToManyField fields so consider removing title from the slug.

Removing title can be justified because slug is not changing and title could be changing these two could eventually become out of sync, oth name could be changed, also...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants