search

compostage-dans-ma-ville / backend

0 stars 0 forks source link

Rendre DailySchedules optionnel dans un Site #39

Closed johannchopin closed 1 year ago

johannchopin commented 1 year ago

Un site n'a pas forcément de DailySchedules (ex: les sites privés à cadenas).

Lyokolux commented 1 year ago

@johannchopin quels sont les critères pour que tu juges ce ticket comme résolu?

johannchopin commented 1 year ago

Juste modifier le schéma prisma et dto j'aurais dit 🤔

Lyokolux commented 1 year ago

L'API retourne alors schedule: null, schedule: undefined ou simplement l'objet sans la propriété schedule ?

Lyokolux commented 1 year ago

Juste modifier le schéma prisma et dto j'aurais dit thinking

Le schéma possède une relation multiple, donc par défaut, un tableau vide est retourné. Cela peut cependant prêter à confusion. null ou undefined serait plus adapté.

Comme les horaires d'ouvertures ne sont pas définies, ou ne sont pas pertinent dans ce cas, undefined serait à préférer :thinking: Null pourrait laisser penser que le site est fermé en permanence.

En revanche, de quoi as-tu besoin pour l'UI?