piair/MyINPulse
2
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

front_foundation #5

Open
mohamed_maoulainine wants to merge 73 commits from front_foundation into main
pull from: front_foundation
merge into: piair:main
Conversation 3 Commits 73 Files Changed 27 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 4ef92efd0e - Show all commits

4
front/MyINPulse-front/src/components/canvas/HeaderCanvas.vue
View File

@ -19,7 +19,7 @@
</template>
mohamed_maoulainine marked this conversation as resolved Outdated
piair commented 2025-02-19 08:57:56 +01:00
Outdated
Review
Copy Link

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

Ce n'est pas de la composition API, a changer
mohamed_maoulainine commented 2025-03-26 11:34:26 +01:00
Outdated
Review
Copy Link

où exactement?

où exactement?
piair commented 2025-02-19 08:55:00 +01:00
Outdated
Review
Copy Link

Il faut se servir de axiosInstance, voir https://gitea.piair.dev/piair/MyINPulse/src/branch/main/front/MyINPulse-front/src/services/api.ts.
Cela permet de préconfigurer l'authentification et le host du backend.

Il faut se servir de `axiosInstance`, voir https://gitea.piair.dev/piair/MyINPulse/src/branch/main/front/MyINPulse-front/src/services/api.ts. Cela permet de préconfigurer l'authentification et le host du backend.
<script setup>
<script setup lang="ts">
import { ref, onMounted } from "vue";
import axios from "axios";
@ -34,7 +34,7 @@ const toggleDropdown = () => {
const fetchEntrepreneurs = async () => {
try {
const response = await axios.get("http://localhost:5000/entrepreneurs");
entrepreneurEmails.value = response.data.map(e => e.email);
entrepreneurEmails.value = response.data.map((e: { email: string }) => e.email);
mohamed_maoulainine marked this conversation as resolved Outdated
piair commented 2025-02-19 08:52:57 +01:00
Outdated
Review
Copy Link

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

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 2025-02-19 09:42:36 +01:00
Outdated
Review
Copy Link

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 serait bien d'appeler mock-data au lieu de fake-data

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 serait bien d'appeler mock-data au lieu de fake-data
} catch (error) {
console.error("Erreur lors de la récupération des entrepreneurs:", error);
}