front_foundation #5
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "front_foundation"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
une 1ere version du front, nécessite l'ajout d'endpoint (en attente d'avancement) pour se connecter à la base.
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).
J'ai laissé pas mal de commentaires, n'hésitez pas à en parler.
@ -5,4 +5,2 @@
<template>
<Header />
<error-wrapper></error-wrapper>
Il faut laisser ça, cela permet d'afficher les messages d'erreurs par desssus le contenu.
@ -17,0 +7,4 @@
<Header />
<RouterLink to="/">Home</RouterLink> |
<RouterLink to="/canvas">Canvas</RouterLink>
Vous êtes sur qu'il faut laisser ça, je pense que ça devrait être inclus dans RouterView
@ -0,0 +1,75 @@
<template>
Je pense que les clients voulaient plus un vrai calendrier genre ADE, mais c'est largement suffisant pour la V1
@ -2,3 +2,3 @@
<div class="project">
<div class="project-header">
<h2>{{ projectName }}</h2>
<h2 @click="goToLink">{{ projectName }}</h2>
Toute la case devrait être clickable, pas que le titre
@ -5,0 +8,4 @@
</div>
<div class="project-body">
<ul>
<li v-for="(name, index) in listName" :key="index">{{ name }}</li>
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.
@ -16,0 +27,4 @@
const goToLink = () => {
if (props.projectLink) {
window.location.href = props.projectLink;
C'est comme un href, a ne pas utiliser
je le corrige après parce que je n'ai pas une idée claire sur comment sera le résultat du request, pour le moment je ferais en sorte de rediriger tout les projets vers le même canvas.
@ -17,0 +58,4 @@
}
/* Button Container */
.project-buttons {
Vérifier que les boutons ne sont pas partageable, ce serait mieux d'utiliser un style commun partout
@ -0,0 +1,87 @@
<template>
Il serait mieux d'afficher le contenu du lean canvas sans devoir clicker dessus, un peu comme sur ce lien.
De plus, ce n'est pas important d'afficher en gros le titre de la section, il devrait être plus discret.
https://cms.boardmix.com/images/articles/lean-canvas.png
En plus il y a des jolies couleur c'est sympa.
@ -0,0 +9,4 @@
</div>
</template>
<script>
ce n'est pas de la composition API
@ -0,0 +10,4 @@
{{ email }}
</button>
</div>
<button class="return-button"><a href="/">Return to list project</a></button>
plutôt utiliser des routerLink au lieu de href
https://stackoverflow.com/questions/52675885/when-to-use-router-link-vs-a
@ -0,0 +17,4 @@
</template>
<script>
Ce n'est pas de la composition API, a changer
où exactement?
@ -0,0 +18,4 @@
<script>
import axios from "axios";
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.
@ -0,0 +24,4 @@
data() {
return {
isDropdownOpen: false,
entrepreneurEmails: [],
Vérifier que le type de données renvoyé par le backend correspond bien à ca (ce n'est pas le cas).
@ -0,0 +34,4 @@
},
async fetchEntrepreneurs() {
try {
const response = await axios.get("http://localhost:5000/entrepreneurs");
il faut pas Hardcoder une addresse, elle est dans le fichier .env. Dans tout les cas, cette adresse ne devrait pas être là.
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
@ -0,0 +10,4 @@
</div>
</template>
<script>
Ce n'est pas de la composition API
@ -0,0 +17,4 @@
}
.produit {
background-color: #f9e4e4;
ce serait cool d'utiliser des variables comme ça https://stackoverflow.com/questions/1875852/how-can-i-define-colors-as-variables-in-css , mais c'est aps urgent du tout
@ -0,0 +108,4 @@
}
/* Dropdown styling */
.contact-dropdown {
Si cela ne concerne que le dropdown d'un contact, cela devrait être dans le style de la view contact
@ -29,6 +29,47 @@ keycloakService.CallInit(() => {
})
// this shit made by me so i can run the canva vue app
svp, des commentaires plus propres...
@ -30,2 +30,4 @@
})
// this shit made by me so i can run the canva vue app
createApp(App).use(router).mount('#app');
Le fait de le laisser ce bout de code casse complètement l'application, c'est dommage. Quand je fais un Makefile qui lance tout les services, c'est justement pour éviter d'avoir besoin de faire ça.
Si vous voulez pas l'utiliser, pas de problème, mais vous pushez pas dans le main.
@ -14,0 +18,4 @@
component: () => import('../views/AdminMain.vue'),
},
// route pour les canvas (made by adnane), in fact the two vue apps are separated for now
c'est en effet a changer pour plus tard, mais ça marche très bien pour le moment.
Pour plus tard il faudra probablement utiliser ça ou ça
@ -0,0 +1,72 @@
<script setup lang="ts">
Il est plus logique de mettre un unique script setup.
Je vais mettre içi une fois pour toute:
je trouve beaucoup plus logique l'ordre
<script>
=><template>
=><style>
que l'ordre qui est utilisé partout dans vos fichiers (
<template>
=><script>
=><style>
)En effet, on utilise dans le template les fonctions qui sont définies dans les scripts, donc c'est plus logique de les définir avant.
C'est ce qui est utilisé par défaut dans le linter (vue-recommanded).
@ -0,0 +13,4 @@
import { ref, onMounted } from "vue";
import axios from "axios";
export default {
composition API
Still some issues
Checkout
From your project repository, check out a new branch and test the changes.