Eco-Gest / Backend-Laravel

0 stars 0 forks source link

refacto user migration #13

Closed AugustinSeguin closed 4 months ago

AugustinSeguin commented 4 months ago
eleonoreeuzenes commented 4 months ago

@AugustinSeguin est-ce que cette story inclus aussi de renvoyer une erreur : if (User::where('username', $request['username'])->count() > 0) { return response()->json([ 'message' => 'Username already used.' ], 409); }

dans le update du usercontroller ?

AugustinSeguin commented 4 months ago

@eleonoreeuzenes non je comptais uniquement rajouter un appel à la fonction unique() pour la migration de la table user et la colonne username, exactement comme ça a été fait sur l'email par exemple. Ce qui fait que si le username existe déjà Eloquent renverra une erreur ;)

On pourrait même supprimer ce bloc de code que tu as cité pour la fonction register

eleonoreeuzenes commented 4 months ago

Ah non faut pas supprimer je l'utilise côté front 😆