• Joined on 2024-01-01
piair pushed to backend-api at piair/MyINPulse 2025-02-26 14:29:44 +01:00
1cebebf1a5 feat: implemented most of the backend api for administrator
d8bc7cc9b6 feat: added log4j. It's way better than System.stderr.
Compare 2 commits »
piair pushed to backend-api at piair/MyINPulse 2025-02-26 10:32:49 +01:00
e66fa33577 feat: added a new database service
piair suggested changes for piair/MyINPulse#5 2025-02-19 13:44:13 +01:00
front_foundation

Still some issues

piair commented on pull request piair/MyINPulse#5 2025-02-19 13:41:59 +01:00
front_foundation

composition API

piair pushed to backend-api at piair/MyINPulse 2025-02-19 12:09:41 +01:00
4698aa549f feat: frontend call now include the token and send the email from the token to backen servie
piair commented on pull request piair/MyINPulse#5 2025-02-19 09:42:36 +01:00
front_foundation

je viens de voir le message de commit, c'est en effet logique d'utiliser cette donnée. Par contre je suis pas sur que ce soit la bonne méthode pour faire des tests, c'est a discuter.

Il…

piair suggested changes for piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Dans l'ensemble c'est pas mal, mais il faut absolument rebase avec master, vérifier que le linter passe (ce qui n'est pas dutout le cas pour le moment).

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

plutôt utiliser des routerLink au lieu de href

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Ce n'est pas de la composition API, a changer

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Vérifier que le type de données renvoyé par le backend correspond bien à ca (ce n'est pas le cas).

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation
piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

il faut pas Hardcoder une addresse, elle est dans le fichier .env. Dans tout les cas, cette adresse ne devrait pas être là.

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Je pense que les clients voulaient plus un vrai calendrier genre ADE, mais c'est largement suffisant pour la V1

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

c'est en effet a changer pour plus tard, mais ça marche très bien pour le moment.

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

ce n'est pas de la composition API

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Il serait mieux d'afficher le contenu du lean canvas sans devoir clicker dessus, un peu comme sur ce lien.

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Ce n'est pas de la composition API

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

C'est comme un href, a ne pas utiliser

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Vérifier que les boutons ne sont pas partageable, ce serait mieux d'utiliser un style commun partout

piair commented on pull request piair/MyINPulse#5 2025-02-19 09:38:53 +01:00
front_foundation

Il faut faire gaffe a ce que va renvoyer l'API, je suis presque sur que vous n'aurez pas une liste de nom directement.