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

LeoMoulin/Backend/Leo #55

Merged
tonitch merged 11 commits from LeoMoulin/Backend/Leo into master 2024-03-05 23:42:00 +01:00
Conversation 0 Commits 11 Files Changed 14 +490
LeoMoulin commented 2024-03-05 18:47:34 +01:00
Owner
Copy Link

Merge d'une première version pour le backend

Merge d'une première version pour le backend
LeoMoulin added 9 commits 2024-03-05 18:47:35 +01:00 
- Ajout des getters/setters appropriés
- Ajout des constructeur nécessaires
- Ajout des champs password et salt dans user
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m25s
Details
Build and test backend / Test-backend (pull_request) Successful in 2m1s
Details
Build and test FrontEnd / Build-frontend (pull_request) Successful in 25s
Details
Build and test FrontEnd / Test-frontend (pull_request) Successful in 23s
Details
c27c3ac9fb 
- Ajout de la table token
- Ajout des foreign keys
Maxime approved these changes 2024-03-05 18:55:50 +01:00
Dismissed
Maxime left a comment
Owner
Copy Link

the rest is ok

the rest is ok
backend/src/main/java/ovh/herisson/Clyde/Tables/Token.java Outdated
@ -0,0 +10,4 @@
@GeneratedValue(strategy = GenerationType.AUTO)
@Id
private int id;
private int regNo;
Maxime commented 2024-03-05 18:54:08 +01:00
Owner
Copy Link

Pourquoi pas directement un User ici ? où y'a pas moyen de faire un link vers la table Users ?

Pourquoi pas directement un User ici ? où y'a pas moyen de faire un link vers la table Users ?
LeoMoulin commented 2024-03-05 20:04:00 +01:00
Author
Owner
Copy Link

J'ai suivi les champs de la table ! J'ai aussi oublié de le mettre en foreign key c'est vrai je vais faire ca j'avoues que j'ai pas pensé a mettre directement une référence vers un objet user j'avoues que je ne sais pas comment ca va se comporter je vais essayer d'approfondir.

J'ai suivi les champs de la table ! J'ai aussi oublié de le mettre en foreign key c'est vrai je vais faire ca j'avoues que j'ai pas pensé a mettre directement une référence vers un objet user j'avoues que je ne sais pas comment ca va se comporter je vais essayer d'approfondir.
Maxime commented 2024-03-05 20:30:20 +01:00
Owner
Copy Link

Ok check comme ça si on peut chopper le User en une seule query sans pour autant stocker une deuxième fois la table User C parfait

Ok check comme ça si on peut chopper le User en une seule query sans pour autant stocker une deuxième fois la table User C parfait
backend/src/main/java/ovh/herisson/Clyde/Tables/Token.java
@ -0,0 +11,4 @@
@Id
private int id;
private int regNo;
private String data;
Maxime commented 2024-03-05 18:54:59 +01:00
Owner
Copy Link

data ? peut-être être plus précis dans le nom ?

data ? peut-être être plus précis dans le nom ?
LeoMoulin commented 2024-03-05 20:04:21 +01:00
Author
Owner
Copy Link

Je vais juste le rename en token comme ca c'est plié !

Je vais juste le rename en token comme ca c'est plié !
tonitch added 1 commit 2024-03-05 20:30:11 +01:00
Merge branch 'master' into LeoMoulin/Backend/Leo
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 1m55s
Details
Build and test backend / Test-backend (pull_request) Successful in 1m52s
Details
Build and test FrontEnd / Build-frontend (pull_request) Successful in 24s
Details
57de070eb1
tonitch dismissed Maxime’s review 2024-03-05 20:30:11 +01:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

Maxime requested changes 2024-03-05 22:37:50 +01:00
backend/src/main/java/ovh/herisson/Clyde/Tables/User.java Outdated
@ -0,0 +32,4 @@
this.birthDate = birthDate;
this.role = role;
this.password = password;
this.salt = salt;
Maxime commented 2024-03-05 22:37:46 +01:00
Owner
Copy Link

Pas Salt !

Pas Salt !
Maxime added 1 commit 2024-03-05 23:41:14 +01:00
correction commit
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 1m53s
Details
Build and test FrontEnd / Build-frontend (pull_request) Successful in 19s
Details
bb0e6783ec
tonitch scheduled this pull request to auto merge when all checks succeed 2024-03-05 23:41:27 +01:00
tonitch merged commit b72c0a8e17 into master 2024-03-05 23:42:00 +01:00
tonitch deleted branch LeoMoulin/Backend/Leo 2024-03-05 23:42:01 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Maxime
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
3 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#55
No description provided.
Delete Branch "LeoMoulin/Backend/Leo"

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?