Skip to content

Best practices Book #599

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

Closed
wants to merge 7 commits into from
Closed

Best practices Book #599

wants to merge 7 commits into from

Conversation

leblanc-simon
Copy link

Import the best practices book and translate the introduction


Nous avons volontairement garder ce guide court. Nous ne voulons par répéter des
explications que vous pouvez trouver dans la vaste documentation de Symfony,
comme les discussions autour de l'injection de dépendance ou des ///front controllers///.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai un gros doute sur comment traduire front controllers en français. Des suggestions ?

///your domain code/// (ex: classes Doctrine) et toute votre logique métier;
* ``vendor/``, c'est le répertoire où Composer installe les dépendances de votre application
et vous ne devez jamais modifier son contenu;
* ``web/``, stocke tous ///the front controller files/// et toutes les ressources web, telles que
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Encore une fois, je n'arrive pas à traduire convenablement "front controller". Des suggestions ?

@gregquat
Copy link
Member

Je crois avoir déjà vu "contrôleur frontal" pour "front controller" (contrôleur front) ça irait aussi.
Vendors ça reste vendor, mais dans le contexte de ta phrase, ça serait plutôt "nom d'organisation" ou un truc du genre.

Attention, je ne peux pas merger cette PR car elle casse le build

@leblanc-simon
Copy link
Author

Merci pour les propositions.

Pour le build, je pense que ce sont les extensions sphinx qu'il faut mettre à jour à la référence e58edd22d16cb247267025d557410dcbfa5fa959
L'import est tiré de la pull request symfony/symfony-docs#4327 et dans les commentaires, pour résoudre le problème de build, il est indiqué (et c'est ce qui a été fait) de mettre à jour la référence.

Je ferai un test et mettrai à jour ma pull request en conséquence

@gregquat
Copy link
Member

Salut, désolé pour le "petit" temps de réaction. Merci pour le travail abattu mais je ne peux pas accepter la PR tant qu'elle contient du texte anglais :/
Ca risque de perturber le référencement naturel sur symfony.com

@leblanc-simon
Copy link
Author

Pas de soucis. Je n'ai pas eu le temps de regarder le reste pour le moment. En fait, j'avais soumis cette PR afin que plus de monde puisse participer. Mais je comprend, la raison.

Je ferme cette issue pour le moment, histoire de ne pas encombrer les issues du projet. Je vais reprendre très prochainement la traduction et je ré-ouvrirai cette PR quand j'aurais pu avancer.

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