kawazrepos/Kommonz — comment on commit 698885dcdf9569baa9bd47c80983c8c2849b2307
File:  line 0
By giginet:

スペースの位置とか、前に指摘した細かいところが直っていて良いですね。

気になった点をいくつか。

models
------------------------
- is_already_readはカラム名としてクドいかも。readでおｋ？
- sent_atは、他のModelがcreated_atなので、多少意味合いが異なるけどそっちの方が無難？（ここは悩みどころ）
- get_absolute_urlには@models.permalinkデコレータを付けましょう。使い方は今度教えます

views
----------------------------
- authorの設定は、Viewレベルではなくて、django-quertのglobal_requestを用いて、Modelレベルで設定させましょう。Materialのcleanメソッドが参考になるので、そのまま写してしまっておｋ
- そもそもこのpost動く？（未確認）
- getがただsuperしてるだけなのが気になるな。後々処理を追加するために仮に書いているだけならいいけど、そうでないなら不要なので書かない方がよさげ
- 同様にMessageDetailViewも書きかけなら良いけど、特に追加の処理がないのであれば、urls.pyに直接書いた方が混乱が少ないと思います

全般
---------------------------
- 同一アプリ内のモジュールを読み出すときは相対参照の方が良いかも。△Kommonz.apps.hoge.fuga ○hoge.fug