search

Karitchi / infolab

0 stars 0 forks source link

Ajout du panneau de configuration , creation endpoint us #68

Closed Ajkll closed 1 week ago

Ajkll commented 1 week ago

Il ne devrait pas y avoir de conflit pour merge cette pr.

La pull request suivante contient les requêtes détaillées dans la user story n°34. Elle contient un nouveau panneau de configuration /config, permettant à l'utilisateur de configurer les heures d'activation depuis l'application. Elle définit un endpoint pour récupérer les informations relatives aux heures d'activation du Raspberry Pi. Cette API est accessible uniquement en "GET"; par exemple : curl http://localhost:3000/api/raspberry-schedule retournera un JSON adapté à l'utilisation de notre Raspberry Pi.

Il serait judicieux d'ajouter une page d'accueil permettant de naviguer vers la page /admin (que je renommerais personnellement /annonce) ou vers la page /config. Ce n'est qu'une suggestion, mais on pourrait aussi envisager de diriger vers :

la page de connexion avec la possibilité de se déconnecter,
une page /historique pour répondre à l'issue où l'on évoquait le besoin d'un historique,
une page pour contrôler les informations affichées, conformément aux besoins mentionnés dans une autre issue.

Note : Spécifier l'heure actuelle devrait éteindre le Raspberry Pi (qui vérifie toutes les 3 minutes pour le moment). Éventuellement, je demanderai à un professeur de me prêter un bouton pour eteindre directement le pi selon le retour des client; M. Dubruille m’a proposé hier de me prêter un Raspberry Pi avec Wi-Fi.

Ajkll commented 1 week ago

Karitchi devrait etre celui qui review cette pr pour la raisson suivante:

Il à accès facilement à l'application en production depuis son compte amazon et cela me permettrait de tester en condition réelle le raspberry pi pour cette tache, le code pour le raspberry serra push demain.

Karitchi commented 1 week ago

La feature est fonctionnelle, aucun bug détecté.

Suggestions d'améliorations :

  1. Afficher les horaires de mise en veille directement dans le tableau des heures (ex. avec une couleur bleue pour indiquer les périodes de veille).
  2. Renommer la table PostgreSQL raspberry_schedules pour clarifier qu'elle stocke les horaires de mise en veille.
  3. La couleur de fond en bas de page est étrange, bien que non liée à cette feature.

Je merge cette version dans main puisqu'elle me parait suffisante 👍