search

twoscoops / two-scoops-of-django-1.8

Tracking thoughts and feature requests for Two Scoops of Django 1.8 in the issue tracker. And the book's code examples are here.
400 stars 81 forks source link

TimeStampedModel #121

Closed maconnell closed 7 years ago

maconnell commented 8 years ago

In 6.1.3 I believe TimeStampedModel should be renamed TimestampedModel.

pydanny commented 7 years ago

The common convention is TimeStampedModel. Any reason why we shouldn't follow it?

maconnell commented 7 years ago

I came across this as a beginner with Django, so hadn't seen that there was a Django convention for "TimeStamp" here. Perhaps it is reasonable to keep this convention.

As a Django beginner I was expecting the usual camel case rules to apply - if I make something from a timestamp, and a model, I get a TimestampModel. But looking at TimeStampedModel it can give the feeling that it's made from a Time and a Stamp, not a timestamp. So, although in this case it was obvious to me that it was a "timestamped model", it wasn't obvious why it wasn't called that. But now I know ;-)

pydanny commented 7 years ago

:smile: