WIP: Leo/Backend/RegNo #147

Closed
LeoMoulin wants to merge 6 commits from Leo/Backend/RegNo into master
Owner

Merge de la fonctionnalité de génération des RegNo

Merge de la fonctionnalité de génération des RegNo
LeoMoulin added 5 commits 2024-03-22 12:07:58 +01:00
Adapt the condition
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m44s
Build and test FrontEnd / Build-frontend (pull_request) Successful in 25s
49b99e14c9
LeoMoulin requested review from Maxime 2024-03-22 12:08:09 +01:00
LeoMoulin requested review from tonitch 2024-03-22 12:08:13 +01:00
LeoMoulin requested review from Wal 2024-03-22 12:08:13 +01:00
LeoMoulin added 1 commit 2024-03-23 18:12:12 +01:00
Delete userSaveCounter and replace it by a static method
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m6s
Build and test FrontEnd / Build-frontend (pull_request) Successful in 24s
c0890ac76a
Maxime approved these changes 2024-03-23 18:43:51 +01:00
Maxime left a comment
Owner

LGTM now 👍 💯

LGTM now 👍 💯
@ -60,1 +58,3 @@
userRepo.saveAll(mockUsers);
mockUsers = new ArrayList<>(Arrays.asList(joke, herobrine, joe, meh, jojo));
userService.saveAll(mockUsers);
userService.save(lena);
Owner

pourquoi pas lena dedans ?

pourquoi pas lena dedans ?
Author
Owner

j'ai fait pleins de petits tests en mode save après saveAll() et inversément pour voir si tout marchait bien je vais tout remettre j'avoues que j'ai zappé

j'ai fait pleins de petits tests en mode save après saveAll() et inversément pour voir si tout marchait bien je vais tout remettre j'avoues que j'ai zappé
Maxime marked this conversation as resolved
@ -108,3 +110,4 @@
public User save(User user){
usc.reset();
user.setPassword(passwordEncoder.encode(user.getPassword()));
return userRepo.save(user);
Owner

pourquoi ne pas re reset ici alors que tu le fais dans le save all ?

pourquoi ne pas re reset ici alors que tu le fais dans le save all ?
@ -113,0 +117,4 @@
RegNoGenerator.resetCount();
userRepo.saveAll(list);
//Reset le compteur a zero pour les futurs ajouts
RegNoGenerator.resetCount();
Owner

pourquoi reset après ?

pourquoi reset après ?
Author
Owner

C'est vrai que ce n'est plus nécessaire je vais le retirer

C'est vrai que ce n'est plus nécessaire je vais le retirer
Maxime marked this conversation as resolved
@ -0,0 +2,4 @@
public final class UserSaveCounter {
private static UserSaveCounter instance;
private int count = 0;
Owner

tu peux juste mettre count en static ici et pas t'emmerder avec un singleton

tu peux juste mettre count en static ici et pas t'emmerder avec un singleton
Maxime marked this conversation as resolved
@ -1,3 +1,2 @@
spring.jpa.hibernate.ddl-auto=create-drop
spring.jpa.database-platform=org.hibernate.dialect.PostgreSQLDialect
spring.sql.init.mode=always
Owner

pourquoi la suppression

pourquoi la suppression
Owner

pourquoi l'avoir retiré ?

pourquoi l'avoir retiré ?
Author
Owner

Aucune idée j'ai surement du faire un trop gros delete je vais le remettre

Aucune idée j'ai surement du faire un trop gros delete je vais le remettre
Maxime marked this conversation as resolved
@ -24,6 +25,7 @@ public class UserRepoTest {
Owner

tu peux faire un test sur les regno ?

tu peux faire un test sur les regno ?
Maxime marked this conversation as resolved
LeoMoulin changed title from Leo/Backend/RegNo to WIP: Leo/Backend/RegNo 2024-03-23 18:52:34 +01:00
LeoMoulin closed this pull request 2024-04-15 20:59:29 +02:00
LeoMoulin deleted branch Leo/Backend/RegNo 2024-04-15 20:59:37 +02:00
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m6s
Build and test FrontEnd / Build-frontend (pull_request) Successful in 24s

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 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#147
No description provided.