search

MAPC / bostonparks

REPOSITORY MOVED TO https://github.com/codeforboston/bostongreenmap
bostongreenmap.org
BSD 3-Clause "New" or "Revised" License
2 stars 2 forks source link

Slug on model class Facility is not unique #96

Closed cspanring closed 12 years ago

cspanring commented 12 years ago

Does it need to be unique?

tyrelsouza commented 12 years ago

I have made it unique now in the models.py file. It would make sense for it to be unique, as a slug is a lot easier on the eyes than someone seeing numbers all over the place and questioning what is being referred.

tyrelsouza commented 12 years ago

Although I'm not sure we even need a slug on Facility, I just checked and it isn't used ANYWHERE in the code. I will do some testing to see what commenting this out affects.

cspanring commented 12 years ago

That's what I thought. There is no facility page where a slug would be useful. We might as well drop it. Just wanted to confirm.

-----Original Message----- From: Tyrel Souza [mailto:reply@reply.github.com] Sent: Wednesday, June 13, 2012 10:46 AM To: Christian Spanring Subject: Re: [bostonparks] Slug on model class Facility is not unique (#96)

Although I'm not sure we even need a slug on Facility, I just checked and it isn't used ANYWHERE in the code. I will do some testing to see what commenting this out affects.

Reply to this email directly or view it on GitHub: https://github.com/MAPC/bostonparks/issues/96#issuecomment-6303018

Please be advised that the Massachusetts Secretary of State considers e-mail to be a public record, and therefore subject to the Massachusetts Public Records Law, M.G.L. c. 66 § 10.

tyrelsouza commented 12 years ago

Yeah, it's commented out.