PGL/Clyde
PGL/Clyde
3
2
Fork 3
Code Issues 13 Pull Requests Actions Projects 3 Releases 2 Wiki Activity

Login et agencement de la main page #51

Merged
Wal merged 9 commits from loginProto into master 2024-03-04 19:42:58 +01:00
Conversation 0 Commits 9 Files Changed 6 +197 -27
Wal commented 2024-03-02 16:01:20 +01:00
Owner
Copy Link
No description provided.
Wal added 6 commits 2024-03-02 16:01:21 +01:00
Merge branch 'master' into loginProto
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 1m54s
Details
Build and test backend / Test-backend (pull_request) Successful in 1m53s
Details
Build and test FrontEnd / Build-frontend (pull_request) Successful in 23s
Details
Build and test FrontEnd / Test-frontend (pull_request) Successful in 22s
Details
c274fc8414
tonitch requested changes 2024-03-02 22:34:54 +01:00
tonitch left a comment
Owner
Copy Link

La page est vraiment stylée!

Pour le coup je suis pas sur que mon commentaire soit claire mais c'est compliqué à expliqué donc n'hésite pas à me demander si tu vois pas.

Pour le coup c'est juste vis à vis de la structure du projet.

La page est vraiment stylée! Pour le coup je suis pas sur que mon commentaire soit claire mais c'est compliqué à expliqué donc n'hésite pas à me demander si tu vois pas. Pour le coup c'est juste vis à vis de la structure du projet.
frontend/login.html Outdated
@ -0,0 +22,4 @@
</div>
</div>
</div>
<script type="module" src="/src/main.js"></script>
tonitch commented 2024-03-02 22:30:45 +01:00
Owner
Copy Link

(*) référencé plus tard

(*) référencé plus tard
frontend/src/assets/main.css Outdated
@ -2,4 +2,3 @@ body {
background-color: rgb(53, 25, 60);
margin:0;
}
tonitch commented 2024-03-02 22:32:59 +01:00
Owner
Copy Link

j'aime bien.

j'aime bien.
frontend/src/main.js Outdated
@ -1,5 +1,5 @@
import './assets/main.css'
import './assets/styleLogin.css'
tonitch commented 2024-03-02 22:32:29 +01:00
Owner
Copy Link

Le fait de mettre le style ici fait que cette feuille de style est chargé pour tout le site (dont la page principale) ce qui pourrait ne pas être désirable.

Le mieux est soit d'utiliser le propre main.js pour la page de login (genre login.js) et de tout refaire dedans (pour ainsi profiter de vue.js dans la page de login)

soit comme c'est le cas maintenant de faire la page en pure html/css sans vue.js et du coup de pas charger ce script et de suprimer le chargement (ligne * référencé). La méthode actuellement n'est pas correcte car elle tente de charger une page Vue dans #app qui n'éxiste pas. (tu peux le voir dans la console quand tu vas sur la page de login)

Le fait de mettre le style ici fait que cette feuille de style est chargé pour tout le site (dont la page principale) ce qui pourrait ne pas être désirable. Le mieux est soit d'utiliser le propre main.js pour la page de login (genre login.js) et de tout refaire dedans (pour ainsi profiter de vue.js dans la page de login) soit comme c'est le cas maintenant de faire la page en pure html/css sans vue.js et du coup de pas charger ce script et de suprimer le chargement (ligne * référencé). La méthode actuellement n'est pas correcte car elle tente de charger une page Vue dans #app qui n'éxiste pas. (tu peux le voir dans la console quand tu vas sur la page de login)
👍 1
tonitch commented 2024-03-02 22:36:58 +01:00
Owner
Copy Link

ps: Si tu souhaite que je clean tout ça moi même ça ne me dérange pas non plus.

ps: Si tu souhaite que je clean tout ça moi même ça ne me dérange pas non plus.
Wal added 1 commit 2024-03-03 18:38:54 +01:00
Convert login.html to Login.vue
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m28s
Details
Build and test backend / Test-backend (pull_request) Successful in 1m53s
Details
Build and test FrontEnd / Build-frontend (pull_request) Successful in 25s
Details
Build and test FrontEnd / Test-frontend (pull_request) Successful in 24s
Details
7087b48245
tonitch requested changes 2024-03-04 13:21:51 +01:00
tonitch left a comment
Owner
Copy Link

ça marche, le seul problème est que tu n'a pas recrée la page login.html donc je l'ai fait sur la branche e66d95335e

Tu peux check sur le lien que je t'ai donné puis si tu veux l'ajouter tu peux aller sur ta branche, faire un git fetch --all pour mettre à jour ma branche puis git cherry-pick e66d95335e7239feac1a8e12dfbdbef6b1ad858c pour intégrer ce commit dans ta branche (puis un push évidement)

ça marche, le seul problème est que tu n'a pas recrée la page login.html donc je l'ai fait sur la branche https://git.herisson.ovh/PGL/Clyde/commit/e66d95335e7239feac1a8e12dfbdbef6b1ad858c Tu peux check sur le lien que je t'ai donné puis si tu veux l'ajouter tu peux aller sur ta branche, faire un `git fetch --all` pour mettre à jour ma branche puis `git cherry-pick e66d95335e7239feac1a8e12dfbdbef6b1ad858c` pour intégrer ce commit dans ta branche (puis un push évidement)
Wal added 1 commit 2024-03-04 13:29:18 +01:00
Fixing the login page availability
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m3s
Details
Build and test backend / Test-backend (pull_request) Successful in 1m51s
Details
Build and test FrontEnd / Build-frontend (pull_request) Successful in 24s
Details
Build and test FrontEnd / Test-frontend (pull_request) Successful in 23s
Details
d774daab59 
To access the login page you need login.html. Then this page loads the
login.js that generate the login.vue in the div#app (with the '#app')
tonitch approved these changes 2024-03-04 13:44:10 +01:00
tonitch left a comment
Owner
Copy Link

lgtm

lgtm
Wal added 1 commit 2024-03-04 19:34:53 +01:00
Merge branch 'master' into loginProto
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m31s
Details
Build and test backend / Test-backend (pull_request) Successful in 1m58s
Details
Build and test FrontEnd / Build-frontend (pull_request) Successful in 21s
Details
a918edd57e
Maxime approved these changes 2024-03-04 19:40:11 +01:00
LeoMoulin approved these changes 2024-03-04 19:42:38 +01:00
Wal merged commit 9850f2145c into master 2024-03-04 19:42:58 +01:00
Wal deleted branch loginProto 2024-03-04 19:42:58 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tonitch
Maxime
LeoMoulin
Labels
Clear labels   
Bug

   
Done

finie, en attente de validation pour ou pret à la fermeture

  
Pas urgent

A faire que si on a le temps

  
Proposition

   
Question

   
TODO

   
Tests

   
URGENT BORDEL DE Q

   
Waiting for review

   
back

   
front
No Label
Bug
Done
Pas urgent
Proposition
Question
TODO
Tests
URGENT BORDEL DE Q
Waiting for review
back
front
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: PGL/Clyde#51
No description provided.
Delete Branch "loginProto"

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?