Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

integratest ok Guillaume Knibiehly #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

geika
Copy link

@geika geika commented May 13, 2022

Bonjour à l'équipe de dev de Mango 3D.
Voici ma version du test.

J'ai installé deux dépendances:

  • Swipper pour simplifier la conception du carousel en cube 3D.
  • React hook form, pour la gestion du formulaire et des erreurs. J'aime bien cette dépendance pour mes gestions de formulaires, parce qu'elle est rapide et performante.

Je n'ai jamais travaillé avec des fichiers webpack custom, et c'est les principale difficultés que j'ai rencontré notamment au niveau des loader pour les css.
Comme il n'y avait pas de spécification pour ces dernier, j'ai fait un fichier classique et je n'ai pas utilisé de framework css.

J'ai également eu des difficultés pour charger des images dans mes components.
Du coup j'ai contourné le problème en les mettant en css (mais j'ai gardé la fonctionnalité du context, j'en ai juste changé ce qu'il me retournait pour continuer à l'utiliser).

J'espère que ce travail vous plaira.

C'était très intéressant, merci.

Au plaisir de vous lire

Guillaume

@leny
Copy link
Member

leny commented May 17, 2022

Bonjour Guillaume.

Désolé du retard, j'étais un peu loin de mes notifications.

On va décortiquer ça au plus vite et on te revient par mail.

@geika
Copy link
Author

geika commented May 19, 2022

Bonjour Leny,

Je reviens vers toi pour te dire que j'ai fait un nouveau commit avec l'ajout de la ligne rouge sur le test.
Par contre je ne sais pas du tout comment faire une nouvelle pull request sur le commit.

Est-ce que tu à accès au commit à tout hasard ?

Cordialement

Guillaume

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.

2 participants