front_foundation #5

Open
mohamed_maoulainine wants to merge 73 commits from front_foundation into main
Collaborator

une 1ere version du front, nécessite l'ajout d'endpoint (en attente d'avancement) pour se connecter à la base.

une 1ere version du front, nécessite l'ajout d'endpoint (en attente d'avancement) pour se connecter à la base.
mohamed_maoulainine added 14 commits 2025-02-19 00:12:53 +01:00
piair requested changes 2025-02-19 09:38:53 +01:00
piair left a comment
Owner

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.

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

Il faut laisser ça, cela permet d'afficher les messages d'erreurs par desssus le contenu.

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

Vous êtes sur qu'il faut laisser ça, je pense que ça devrait être inclus dans RouterView

Vous êtes sur qu'il faut laisser ça, je pense que ça devrait être inclus dans RouterView
@ -0,0 +1,75 @@
<template>
Owner

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

Je pense que les clients voulaient plus un vrai calendrier genre ADE, mais c'est largement suffisant pour la V1
mohamed_maoulainine marked this conversation as resolved
@ -2,3 +2,3 @@
<div class="project">
<div class="project-header">
<h2>{{ projectName }}</h2>
<h2 @click="goToLink">{{ projectName }}</h2>
Owner

Toute la case devrait être clickable, pas que le titre

Toute la case devrait être clickable, pas que le titre
mohamed_maoulainine marked this conversation as resolved
@ -5,0 +8,4 @@
</div>
<div class="project-body">
<ul>
<li v-for="(name, index) in listName" :key="index">{{ name }}</li>
Owner

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.

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

C'est comme un href, a ne pas utiliser

C'est comme un href, a ne pas utiliser
Author
Collaborator

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.

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.
piair marked this conversation as resolved
@ -17,0 +58,4 @@
}
/* Button Container */
.project-buttons {
Owner

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

Vérifier que les boutons ne sont pas partageable, ce serait mieux d'utiliser un style commun partout
@ -0,0 +1,87 @@
<template>
Owner

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.

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

ce n'est pas de la composition API

ce n'est pas de la composition API
mohamed_maoulainine marked this conversation as resolved
@ -0,0 +10,4 @@
{{ email }}
</button>
</div>
<button class="return-button"><a href="/">Return to list project</a></button>
Owner

plutôt utiliser des routerLink au lieu de href
https://stackoverflow.com/questions/52675885/when-to-use-router-link-vs-a

plutôt utiliser des routerLink au lieu de href https://stackoverflow.com/questions/52675885/when-to-use-router-link-vs-a
mohamed_maoulainine marked this conversation as resolved
@ -0,0 +17,4 @@
</template>
<script>
Owner

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

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

où exactement?

où exactement?
mohamed_maoulainine marked this conversation as resolved
@ -0,0 +18,4 @@
<script>
import axios from "axios";
Owner

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.
@ -0,0 +24,4 @@
data() {
return {
isDropdownOpen: false,
entrepreneurEmails: [],
Owner

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

Vérifier que le type de données renvoyé par le backend correspond bien à ca (ce n'est pas le cas).
mohamed_maoulainine marked this conversation as resolved
@ -0,0 +34,4 @@
},
async fetchEntrepreneurs() {
try {
const response = await axios.get("http://localhost:5000/entrepreneurs");
Owner

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à.
Owner

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
mohamed_maoulainine marked this conversation as resolved
@ -0,0 +10,4 @@
</div>
</template>
<script>
Owner

Ce n'est pas de la composition API

Ce n'est pas de la composition API
mohamed_maoulainine marked this conversation as resolved
@ -0,0 +17,4 @@
}
.produit {
background-color: #f9e4e4;
Owner

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

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
mohamed_maoulainine marked this conversation as resolved
@ -0,0 +108,4 @@
}
/* Dropdown styling */
.contact-dropdown {
Owner

Si cela ne concerne que le dropdown d'un contact, cela devrait être dans le style de la view contact

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
Owner

svp, des commentaires plus propres...

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');
Owner

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.

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
Owner

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

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](https://vuejs.org/guide/components/provide-inject) ou [ça](https://stackoverflow.com/a/66891577)
@ -0,0 +1,72 @@
<script setup lang="ts">
Owner

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

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).
adnane added 1 commit 2025-02-19 11:02:19 +01:00
adnane added 1 commit 2025-02-19 12:15:47 +01:00
piair reviewed 2025-02-19 13:41:59 +01:00
@ -0,0 +13,4 @@
import { ref, onMounted } from "vue";
import axios from "axios";
export default {
Owner

composition API

composition API
mohamed_maoulainine marked this conversation as resolved
piair requested changes 2025-02-19 13:44:13 +01:00
piair left a comment
Owner

Still some issues

Still some issues
mohamed_maoulainine added 2 commits 2025-02-25 22:53:52 +01:00
mohamed_maoulainine added 1 commit 2025-02-26 03:40:10 +01:00
piair closed this pull request 2025-02-28 09:03:40 +01:00
piair reopened this pull request 2025-02-28 09:03:46 +01:00
adnane added 1 commit 2025-03-19 09:48:08 +01:00
adnane added 1 commit 2025-03-19 10:52:16 +01:00
adnane added 1 commit 2025-03-19 11:54:37 +01:00
adnane added 1 commit 2025-03-23 21:59:32 +01:00
adnane added 1 commit 2025-03-26 11:07:55 +01:00
adnane added 1 commit 2025-03-26 11:31:02 +01:00
mohamed_maoulainine added 8 commits 2025-03-26 11:33:04 +01:00
adnane added 7 commits 2025-03-26 11:57:17 +01:00
mohamed_maoulainine added 1 commit 2025-03-26 11:59:36 +01:00
fix: Adnane?
Some checks failed
CI / build (push) Failing after 8s
b9f3bbbe15
mohamed_maoulainine added 1 commit 2025-04-01 01:39:11 +02:00
Merge branch 'main' into front_foundation
Some checks failed
Format / formatting (push) Successful in 6s
Build / build (push) Successful in 39s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
7c271d8c47
adnane added 6 commits 2025-04-02 08:44:29 +02:00
feat: added openapi documentation to the project
All checks were successful
CI / build (push) Successful in 11s
ef8c8e896d
docs: added needed modications and reorganized yaml file
All checks were successful
CI / build (push) Successful in 11s
b355463dd9
docs: added examples and changed some weird character
All checks were successful
CI / build (push) Successful in 11s
067eeb9494
merging
Some checks failed
Format / formatting (push) Successful in 7s
Build / build (push) Successful in 39s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 6s
3dc8131c33
adnane added 1 commit 2025-04-02 10:52:43 +02:00
fix: manine try now
Some checks failed
Format / formatting (push) Successful in 6s
Build / build (push) Successful in 38s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
00a733c03b
adnane added 1 commit 2025-04-02 11:51:43 +02:00
feat: mode admin added 'first try'
Some checks failed
Format / formatting (push) Successful in 6s
Build / build (push) Successful in 37s
CI / build (push) Failing after 7s
Format / formatting (pull_request) Successful in 5s
03897e1139
mohamed_maoulainine added 2 commits 2025-04-05 21:10:38 +02:00
Merge branch 'front_foundation' of https://gitea.piair.dev/piair/MyINPulse into front_foundation
Some checks failed
Format / formatting (push) Successful in 8s
Build / build (push) Successful in 43s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
0c724cae7f
mohamed_maoulainine added 1 commit 2025-04-06 02:02:50 +02:00
ongoing fix: working through the auth issues, we got the test working again but the requests now give an error code 403
Some checks failed
Format / formatting (push) Successful in 6s
Build / build (push) Successful in 39s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 8s
2b1666c949
mohamed_maoulainine added 1 commit 2025-04-08 23:16:50 +02:00
fix: app
Some checks failed
Format / formatting (push) Successful in 7s
Build / build (push) Successful in 39s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
2dfee66958
adnane added 1 commit 2025-04-09 00:05:20 +02:00
feat: enhancing visuals
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
647812576e
adnane added 1 commit 2025-04-09 00:20:05 +02:00
feat: still enhancing the visuals
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
CI / build (push) Has been cancelled
Format / formatting (pull_request) Successful in 5s
8c4b9ceb9d
mohamed_maoulainine added 2 commits 2025-04-09 00:20:56 +02:00
Merge branch 'front_foundation' of https://gitea.piair.dev/piair/MyINPulse into front_foundation
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
b9647ce36e
mohamed_maoulainine added 1 commit 2025-04-09 00:57:30 +02:00
feat: added form to add project and pending project section for admin, i forgot to push them
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
60290956ec
adnane added 1 commit 2025-04-11 19:42:42 +02:00
feat: using bootstrap..
Some checks failed
Format / formatting (push) Successful in 7s
Build / build (push) Successful in 43s
CI / build (push) Failing after 11s
Format / formatting (pull_request) Successful in 6s
f8991e90ab
mohamed_maoulainine added 1 commit 2025-04-12 03:55:54 +02:00
feat: added an login page but the auth issue is still persisting, found a way to interprete the token the line is commneted in case it breaks code or needs a change
Some checks failed
Format / formatting (push) Successful in 6s
Build / build (push) Successful in 39s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
2b31753265
mohamed_maoulainine added 1 commit 2025-04-12 22:22:46 +02:00
fix: fixed an issue with the server by adding the dependency of jwt
Some checks failed
CI / build (push) Waiting to run
Format / formatting (push) Successful in 6s
Build / build (push) Has been cancelled
Format / formatting (pull_request) Successful in 6s
70658e4fb9
adnane added 2 commits 2025-04-12 22:23:27 +02:00
Merge branch 'front_foundation' of https://gitea.piair.dev/piair/MyINPulse into front_foundation
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
Format / formatting (pull_request) Successful in 5s
CI / build (push) Failing after 8s
ad1fd45bed
mohamed_maoulainine added 2 commits 2025-04-14 01:33:44 +02:00
mohamed_maoulainine added 1 commit 2025-04-15 23:13:13 +02:00
fix: typo
Some checks failed
Format / formatting (push) Successful in 8s
Build / build (push) Successful in 47s
CI / build (push) Failing after 12s
Format / formatting (pull_request) Successful in 6s
c4ba7646d5
adnane added 4 commits 2025-04-16 10:37:05 +02:00
Merge branch 'front_foundation' of https://gitea.piair.dev/piair/MyINPulse into front_foundation
Some checks failed
Format / formatting (push) Successful in 6s
Build / build (push) Successful in 37s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
ba99b3c2b0
adnane added 1 commit 2025-04-16 10:59:28 +02:00
feat: adding signup page
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 5s
6b3cb2610d
mohamed_maoulainine added 1 commit 2025-04-16 11:59:13 +02:00
fix: adminview
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 7s
8b863ee4b1
Some checks failed
Format / formatting (push) Successful in 5s
Build / build (push) Successful in 38s
CI / build (push) Failing after 8s
Format / formatting (pull_request) Successful in 7s
This pull request doesn't have enough required approvals yet. 0 of 3 official approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin front_foundation:front_foundation
git checkout front_foundation
Sign in to join this conversation.
No Reviewers
No Label
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: piair/MyINPulse#5
No description provided.