• Joined on 2024-01-01
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

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

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 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

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

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
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

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

plutôt utiliser des routerLink au lieu de href

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 pushed to backend-api at piair/MyINPulse 2025-02-18 22:15:45 +01:00
04a73073c1 feat: switch to a service layer architecture
piair pushed to backend-api at piair/MyINPulse 2025-02-18 17:20:12 +01:00
11ab5e1dc4 feat: merged Keycloak / postgres
820757c836 fix: corrected formatter error
730aa5f450 Merge remote-tracking branch 'refs/remotes/origin/back-postgres' into back-postgres
dda3e5fcfd fix: most likely fixed merge conflict
5e8e875a37 feat: added user deletion and custom api call in the frontend
Compare 10 commits »
piair deleted branch back-keycloak from piair/MyINPulse 2025-02-18 17:00:08 +01:00
piair closed pull request piair/MyINPulse#4 2025-02-18 16:59:33 +01:00
back-keycloak
piair pushed to back-postgres at piair/MyINPulse 2025-02-18 16:59:23 +01:00
820757c836 fix: corrected formatter error
piair pushed to back-postgres at piair/MyINPulse 2025-02-18 16:56:38 +01:00
730aa5f450 Merge remote-tracking branch 'refs/remotes/origin/back-postgres' into back-postgres
dda3e5fcfd fix: most likely fixed merge conflict
5e8e875a37 feat: added user deletion and custom api call in the frontend
86e7dc7c75 fix: removed the test file that was causing the linter to fail
6235fe7e68 feat: separated class definition
Compare 6 commits »
piair created pull request piair/MyINPulse#4 2025-02-18 16:50:01 +01:00
back-keycloak