Update remove.rst #3924
Update remove.rst #3924
Conversation
Add note about removing web/bundles/acmedemo directory
| @@ -73,6 +73,12 @@ should remove the bundle from the filesystem. The bundle is located in the | |||
| ``src/Acme/DemoBundle`` directory. You should remove this directory and you | |||
| can remove the ``Acme`` directory as well. | |||
|
|
|||
| 3.1 Remove bundle resources | |||
There was a problem hiding this comment.
this new section should be moved below the tip box
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
|
||
| Remove the Acme resource directory from the application's web folder. It can be found in ``web/bundles/acmedemo``. | ||
|
|
There was a problem hiding this comment.
I think we can be even more direct. What do you guys think about:
Remove the
web/bundles/acmedemodirectory. This contains the public assets (e.g CSS) for the bundle.
There was a problem hiding this comment.
I think we should also take care of the second goal of this article: talk about removing a bundle in general. So I would propose something along the lines of "Remove the assets of the bundle in the web/ directory (e.g. web/bundles/acmedemo)"
|
Ping @Ha99y! What do you think about my suggestion? Do you have some time to make the change? Just let me know if you don't :). Thanks! |
| @@ -81,6 +81,11 @@ can remove the ``Acme`` directory as well. | |||
|
|
|||
| echo $this->container->get('kernel')->getBundle('AcmeDemoBundle')->getPath(); | |||
|
|
|||
| 3.1 Remove Bundle Resources | |||
There was a problem hiding this comment.
I would say "Assets" instead of "Resources"
|
Hi @Ha99y, do you have some time to make the changes proposed by the doc reviewers? If not, please feel free to say it and we'll take care of this. Thank you! |
|
Replaced by #4238 |
Add note about removing web/bundles/acmedemo directory