Skip to content

Test Zach#2

Open
Zach-Fisherman wants to merge 1 commit into
whatson-web:masterfrom
Zach-Fisherman:master
Open

Test Zach#2
Zach-Fisherman wants to merge 1 commit into
whatson-web:masterfrom
Zach-Fisherman:master

Conversation

@Zach-Fisherman
Copy link
Copy Markdown

No description provided.

@cyril-whatson
Copy link
Copy Markdown
Contributor

Revue du test

Bonjour Zach,

Merci d'avoir effectué la mission sur ce module d'évaluation Symfony.

Voici un petit retour, pourrais tu effectuer les corrections / améliorations suivantes :

  • Créer les fichiers de migrations
    https://symfony.com/doc/current/doctrine.html

  • Warning lors de l'import des fixtures :
    Warning: Use of undefined constant ADMIN_ROLE - assumed 'ADMIN_ROLE' (this will throw an Error in a future version of PHP)

  • Erreurs suivantes :

    • Problème sur l'entité UploadCSV
      No identifier/primary key specified for Entity "App\Entity\UploadCSV". Every Entity must have an identifier/primary key.

    • Le repository déclaré dans l'entité UploadCSV est incorrect

  • Développer l'import des données fichier csv place.csv en base de données

  • Ajouter des restrictions d'accès si l'utilisateur n'est pas connecté
    Si l'utilisateur n'est pas connecté, il ne doit pouvoir accéder qu'aux pages login/register
    https://symfony.com/doc/current/security.html

N'hésites pas à revenir vers nous si tu as besoin de plus d'informations.

A très vite :)
Cyril

@Zach-Fisherman
Copy link
Copy Markdown
Author

Zach-Fisherman commented Apr 10, 2019 via email

@Zach-Fisherman
Copy link
Copy Markdown
Author

Zach-Fisherman commented Apr 16, 2019 via email

@cyril-whatson
Copy link
Copy Markdown
Contributor

cyril-whatson commented Apr 16, 2019

Hello Zach,

Tu peux regarder la librairie league/csv qui facilite la manipulation des fichiers CSV.

Tu trouveras les informations ainsi que la documentation sur ce bundle ici

N'hésite pas à revenir vers moi si tu as d'autres questions

A bientôt,

Cyril

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants