feat/login #10
Reference in New Issue
Block a user
Delete Branch "feat/login"
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?
pour relecture :)
@@ -10,1 +9,4 @@KAZ_API_PASSWORD=###> symfony/framework-bundle ###APP_ENV=devLes variables APP_SECRET, APP_ENV... sont en plusieurs exemplaires dans le fichier .env
je ne sais pas pourquoi des données se sont ajoutées, je n'ai rien touché :S
@@ -0,0 +1,3 @@# define your env variables for the test env hereKERNEL_CLASS='App\Kernel'Pourquoi avoir fait un fichier ".env.test" ? il semble indiquer qu'il y a des tests unitaires mais il n'y en a pas dans les sources
je ne sais pas pourquoi des données se sont ajoutées, je n'ai rien touché :S
@@ -36,0 +50,4 @@###< symfony/framework-bundle ######> phpunit/phpunit ###/phpunit.xmlencore des lignes en double par exemple phpunit.xml ;)
je ne sais pas pourquoi des données se sont ajoutées, je n'ai rien touché :S
@@ -0,0 +1,18 @@Pourquoi avoir ajouté un fichier compose.override.yaml ? il apporte quoi de plus que le fichier compose.yaml ?
@@ -27,2 +27,4 @@MP_SMTP_AUTH_ALLOW_INSECURE: 1###> doctrine/doctrine-bundle ###database:Section database en double, créer un problème lors du lancement des dockers en local :)
@@ -1,6 +1,12 @@security:# Hierarchie des rôles #role_hierarchy:ROLE_ORGANISATION: ROLE_USER2x le rôle user ?
une erreur d’inattention ^^"
normalement, ça aurait du être : ROLE_ADMIN: ROLE_ADMIN
@@ -13,2 +15,2 @@#[Route(path: '/login', name: 'app_login', methods: ['GET','POST'])]public function login(AuthenticationUtils $authenticationUtils): Response#[Route(path: '/test-password', name: 'test_password')]public function testPassword(UserRepository $userRepo, UserPasswordHasherInterface $hasher): ResponseMéthode avec le nom "testPassword" !!! Elle fait quoi de beau cette methode ? Le nom n'est pas très clair :)
C'était une route test, mais je ne l'ai plus dans mon code, donc je ne comprends pas que tu l'aies encore :S
@@ -15,1 +16,4 @@public function testPassword(UserRepository $userRepo, UserPasswordHasherInterface $hasher): Response{// 1. On va chercher ton admin directement en base$admin = $userRepo->findOneBy(['email' => 'admin@kaz.fr']);L'email de l'utilisateur devrait être passé en paramètre non ?
Je n'ai plus cette route dans mon code, donc je ne comprends pas que tu l'aies encore :S
@@ -38,0 +40,4 @@# #[Route('/user/{email}', name: 'app_user', methods: ['GET'])]# public function index(string $email, KazApiService $apiClient): Response# {# $exist = $apiClient->getUserData($email);On va pouvoir regarder comment fonctionne l'api :) et réactiver cette route
tout à fait ! pour l'instant, je l'ai ignorée volontairement et je me suis basée sur mes fixtures pour vérifier que ma mise en page était OK
@@ -38,0 +69,4 @@if ($form->isSubmitted() && $form->isValid()) {/** @var UploadedFile $imageFile */$imageFile = $form->get('image')->getData();Le formulaire permet seulement de modifier l'image ?
normalement, non. Je peux modifier l'image, le numéro de téléphone et l'adresse mail de secours
@@ -43,0 +117,4 @@# Vérification du formulaire, s'il est bien soumis et valideif ($form->isSubmitted() && $form->isValid()) {# Récupération des données du formulaire$user = $this->getUser();On a deux récupération de l'utilisateur
$user = $this->getUset()je me suis trompée
@@ -43,0 +121,4 @@$plainOldPassword = $form->get('oldPassword')->getData();$newPassword = $form->get('newPassword')->getData();# Vérification de l'ancien mot de passeTrès bien de vérifier l'ancien mot de passe
@@ -29,20 +29,15 @@ class AppFixtures extends Fixturefor ($i = 0; $i < 10; $i++) {Niquel les fixtures :)
@@ -36,3 +38,3 @@#[ORM\Column(length: 255)]private ?string $emailQuota = null;private ?string $emailQuota = '1G';Pourquoi emailQuota à une valeur par défaut et pas les autres quotas ? Comm nextcloudQuota par exemple ?
parce que papa m'a dit qu'un adhérent à un espace de 1G max pour les mails, qu'importe l'offre choisie
@@ -51,3 +54,3 @@#[ORM\Column(length: 255)]private ?string $nextcloudQuota = '1G';private ?string $nextcloudQuota = null;Pourquoi mettre une valeur par défault sur l'email et non sur le nextcloud ?
parce que la taille du cloud dépend de l'offre choisie par l'adhérent
@@ -0,0 +1,65 @@<nav class="bg-white border-b border-gris-clair shadow-sm py-4 px-6 sticky top-0 z-50 font-sora">Niquel les templates et le code front :)
@@ -0,0 +42,4 @@</li>{# Onglet : Mon organisation (ne s'affiche que si on a le rôle adéquat) #}{% if is_granted('ROLE_ADMIN_ORGANISATION') %}Top de conditionner l'affichage selon le rôle de l'utilisateur :)
@@ -0,0 +17,4 @@</h2>{# Zone réservée pour les futures données de l'APITODO : Gérer les données de l'API #}Bonne pratique de mettre les TODO dans le code :)
@@ -40,0 +48,4 @@</div>{# checkbox "se souvenir de moi" #}<div class="flex items-center pt-1">A ton avis ? comment on pourrais mettre en place la fonctionnalité "se souvenir de moi" ? ou est-ce que cette fonctionnalité se joue dans le code backend de symfony ?
j'aurais du faire ça dans le controller plutôt ?
@@ -3,0 +7,4 @@{{ form_start(form, {'attr': {'class': 'max-w-6xl mx-auto px-4 sm:px-6 lg:px-8 grid md:grid-cols-3 gap-8'}}) }}{# TODO: voir si c'est pertinent avec l'API et s'il y a l'utilité d'une photo de profil #}On peut aussi avoir l'information de la photo seulement au niveau de l'app symfony :)
C'est toujours sympa d'avoir de la personnalisation pour les utilisateurs
d'ac !
profil_infos.html.twig, mise à jour des formulaires, meilleur affichage des quotas et permissions, et support des fichiers GIF) 7a25779c9cView command line instructions
Checkout
From your project repository, check out a new branch and test the changes.