-
-
Notifications
You must be signed in to change notification settings - Fork 630
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
mv spatie/symfony-ignition-bundle/0.0/config/packages{,/dev}/ignition.yaml #1689
Conversation
…on" (in "config/packages/ignition.yaml"). Looked for namespace "ignition", found "framework", "doctrine", "twig".` when under environments other than `dev`
Thanks for the PR 😍 How to test these changes in your application
Diff between recipe versionsIn order to help with the review stage, I'm in charge of computing the diff between the various versions of patched recipes. |
…ntrib#1689 and partial revert af67ff2 for `config/bundles.php` * disable doctrine query cache to fix https://stackoverflow.com/questions/30871721/doctrine2-dynamic-table-name-for-entity#comment139425159_49796901 @ `config/packages/doctrine.yaml` * rename var `APP_BASE_URL_ASSETS` to `APP_BASE_URL_BE`, also affect its only usage in `config/packages/framework.yaml` * set the default environment to production @ .env @ be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @n0099, thank you for your contribution!
I would suggest an alternative approach to wrap config with when@dev
like so (instead of moving config to config/packages/dev
):
when@dev:
ignition:
application_path: '%kernel.project_dir%'
dark_mode: false
should_display_exception: '%kernel.debug%'
This approach is available since symfony 5.3 and spatie/symfony-ignition-bundle
minimal requirement is 5.4, so we should be safe to use it.
And it's currently preferred way to do it, as shown in example https://github.com/symfony/demo/blob/f39ffa0c5983cd50f73a8d6ef1026422ca4ec355/config/packages/monolog.yaml#L5-L24
I've opened PR with same fix at spatie/symfony-ignition-bundle#29
Head branch was pushed to by a user without write access
…on.yaml` @ be
fix
There is no extension able to load the configuration for "ignition" (in "config/packages/ignition.yaml"). Looked for namespace "ignition", found "framework", "doctrine", "twig".
when under environments other thandev