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

Book/http fundamentals patch #785

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

anivon
Copy link

@anivon anivon commented Oct 18, 2015

Add of a missing part from english doc and some corrections

@@ -518,7 +521,7 @@ des emails, valider des entrées d'utilisateurs et gérer la sécurité.

La bonne nouvelle est qu'aucun de ces problèmes n'est unique. Symfony fournit
un framework rempli d'outils qui vous permettent de construire votre
application, mais pas vos outils. Avec Symfony2, rien ne vous est imposé :
application, pas vos outils. Avec Symfony2, rien ne vous est imposé :
Copy link
Member

Choose a reason for hiding this comment

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

Petit espace en trop, tu peux aussi laisser Symfony sans le 2 vu qu'on approche de Symfony 3 qui bénéfinira de la même doc !

Copy link
Author

Choose a reason for hiding this comment

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

Ok ! C'est noté ! Je modifie tout ça rapidement

@Nek-
Copy link
Member

Nek- commented Oct 25, 2015

👍 merci

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