-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Il faut entrer nos informations de users dans Remboursements??? #9
Comments
Également dans sa devrait plutôt etre: var RequestSchema = new Schema({ |
Ouais, j'devais faire un merge avec master pour régler ça. (D'où mon pull request initial) Pour ton deuxième point, j'suis pas encore assez à l'aise avec Mongo pour tester, mais je voulais que les données du user dans sa demande soient figées dans la demande. |
Tu devrais enlever toute la section sur les données du user de ton formulaire. Et dans le back-end simplement ajouter l'Id du user actuelle dans la requête. Pourquoi les données de l'utilisateur doivent être figées? Selon moi, si un usager change ses informations personnels ceux-ci devrait également être updater dans la demande. Tu pourrait aller chercher les informations du user avec la rêquete suivante: |
Je suis d'accord que si le user met ses infos à jour, elles devraient se mettre à jour dans une demande active, mais pour les archives, stocker les informations de manière statique m'avait l'air d'être la meilleure option. Pour le stockage des données, j'y reviendrai quand je serai rendu-là. |
Si on veut garder l'information statique je suggère de faire un schema RequestArchive ou les requêtes compléter pourrait être déplacer. Cependant, je ne vois pas vraiment pourquoi garder les informations statiques dans les archives est important. Les informations du user ne sont pas critiques et nous sommes certains qu'au final le cip ne changera pas! |
Je vien de remarquer que le users doit entrer ses informations personnels dans remboursements... Pourtant toutes ses informations se retrouve dans req.session.userInfo.
The text was updated successfully, but these errors were encountered: