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

Update buisness-logic #7025

Closed
wants to merge 2 commits into from
Closed

Update buisness-logic #7025

wants to merge 2 commits into from

Conversation

zairigimad
Copy link
Contributor

Replace all folders names with symfony3-project instead of symfony2-project because symfony 2 and 3 don't have the same structure of folders .

Replace all folder names with symfony3-project instead of  symfony2-project because symfony 2 and 3 don't have the same structure of folders .
@@ -35,7 +35,7 @@ and put things there:

.. code-block:: text

symfony2-project/
symfony3-project/
Copy link

Choose a reason for hiding this comment

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

Why not just call it symfony-project?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes it will be generic !

Changing symfony2-project to symfony-project
@snoek09
Copy link

snoek09 commented Oct 3, 2016

Thanks @zairigimad 👍

Status: reviewed.

@javiereguiluz
Copy link
Member

👍

1 similar comment
@wouterj
Copy link
Member

wouterj commented Oct 3, 2016

👍

@wouterj
Copy link
Member

wouterj commented Oct 6, 2016

Thanks Zairig, nice improvement! I've merged it into the 2.7 version of the docs (the oldest version we still maintain) and I'll merge it into the newer versions from there.

@wouterj wouterj closed this in 79b59a0 Oct 6, 2016
@zairigimad
Copy link
Contributor Author

@wouterj Welcome , i'm happy to contribute in my favorite framework 👍

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

Successfully merging this pull request may close these issues.

None yet

5 participants