Delete some remarks.
This commit is contained in:
parent
ab511d7494
commit
c7635e1b86
|
@ -13,28 +13,6 @@ class Message(models.Model):
|
||||||
is_read (bool): Indique si le message a été lu ou non - par défaut: False
|
is_read (bool): Indique si le message a été lu ou non - par défaut: False
|
||||||
message_title (str): Titre ou sujet du message.
|
message_title (str): Titre ou sujet du message.
|
||||||
message_body (text): Contenu du message.
|
message_body (text): Contenu du message.
|
||||||
|
|
||||||
Remarks:
|
|
||||||
. Les champs `message_body` et `message_title` me posent problème,
|
|
||||||
parce qu'on est déjà dans la classe `Message`. Pourquoi pas juste `title` et `body` ?
|
|
||||||
|
|
||||||
. Le champ `is_read` est superflu. Si on veut savoir si le message est lu, il suffit de
|
|
||||||
faire une méthode avec un décorateur @property et retourner si `date_of_reading` est vide.
|
|
||||||
|
|
||||||
. Par convention, j'aurais appelé les champs `writer` et `reader` plutôt `sender` et `recipient`.
|
|
||||||
|
|
||||||
. Pareil pour les champs "date" auto-remplis. J'aime bien les conventions type `<nom_du_champ>_at`.()
|
|
||||||
(Ici, `written_at` et `read_at`).
|
|
||||||
|
|
||||||
. Est-ce qu'un message ne peut pas être envoyé à plusieurs personnes en même temps ?
|
|
||||||
|
|
||||||
. Tes related_names ne sont pas corrects: `have_write` et `have_read` n'ont pas de signification.
|
|
||||||
Quand on a une instance d'un utilisateur en main, ça reviendrait à faire `user_instance.have_read.all`.
|
|
||||||
Sémantiquement, c'est pas top. Il faudrait plutôt les appeler `sent_messages` et `received_messages`.
|
|
||||||
Dans le code, cela donnerait `user_instance.sent_messages.all()` ou `user_instance.sent_messages.filter(...)`.
|
|
||||||
|
|
||||||
. Est-ce qu'un `Message` ne devrait pas être `Markdownizable` ?
|
|
||||||
Mais alors le champ `information` devrait s'appeler `content` ou quelque chose de plus générique.
|
|
||||||
"""
|
"""
|
||||||
|
|
||||||
writer = models.ForeignKey(
|
writer = models.ForeignKey(
|
||||||
|
|
|
@ -5,11 +5,6 @@ from django.urls import path, re_path
|
||||||
from . import views
|
from . import views
|
||||||
|
|
||||||
|
|
||||||
""" NOTE: Attention à bien nommer les URLs. `message_sent` -> `sent_messages`...
|
|
||||||
|
|
||||||
"""
|
|
||||||
|
|
||||||
|
|
||||||
message_urlpatterns = [
|
message_urlpatterns = [
|
||||||
path(r"sent/", views.get_message_sent, name="message_sent"),
|
path(r"sent/", views.get_message_sent, name="message_sent"),
|
||||||
path(r"received/", views.get_message_received, name="message_received"),
|
path(r"received/", views.get_message_received, name="message_received"),
|
||||||
|
|
Loading…
Reference in New Issue