alyrow / cyrel

Apache License 2.0
5 stars 0 forks source link

cyrel-sync: utiliser plus d'un `event` par type #6

Open luc65r opened 2 years ago

luc65r commented 2 years ago

https://github.com/alyrow/cyrel/blob/master/cyrel-sync/src/main.rs#L246 Pour l'instant, on envoie seulement le dernier event par type (ex : prof ou salle pour un cours) à la DB.

Il y a trois façons de stocker plusieurs event dans la DB :

La seconde me parait mieux, et plus propre.

Il se pose aussi la question du format d'envoi JSON : est-ce qu'il faudrait envoyer les informations dans une chaîne formatée ? dans un tableau ? dans un tableau sauf si il n'y a qu'une seule entrée ?

alyrow commented 2 years ago

Je suis aussi en faveur de la deuxième option qui permettrait d'utiliser les fonctionnalités de la DB mais surtout à l'avenir de pouvoir implémenter un système de recherche.

Le mieux serait alors lors de l'envoi au format JSON, d'envoyer les informations dans un tableau même s'il y a une seule entrée (mieux pour le backend web en tout cas).

luc65r commented 2 years ago

@hcaumeil tu veux implémenter ça ?