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

fix composer.json to allow easier installation #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fritzmg
Copy link

@fritzmg fritzmg commented Mar 31, 2019

This PR provides the following changes:

  • Removed symfony/framework-bundle dependency, since no component of this bundle is actually used. This enables easier installation of the extension in Contao >4.4.
  • Added missing dependencies of actually used components.
  • Removed leofeyer/optimize-native-functions-fixer dev dependency since that is already integrated in the PHP CS Fixer.
  • Removed unused dev dependencies.
  • Fixed indentation.
  • Change license notation to its official format.

Unrelated question: why do you have your own ExceptionConverterListener? Is there any specific reason for that? Otherwise you should just simply throw all the Contao\CoreBundle\Exception\… exceptions directly.

@reluem
Copy link

reluem commented Sep 17, 2019

Please merge this @hschottm. Your version of symfony/framework-bundle clashes with contao 4.8

@klimeckt
Copy link

@hschottm
Hi Helmut, jus upgraded my sites from 4.4 to 4.9.2 and cannot install the faqextensions. Is there any planning to upgrade this extension?
Best, Thomas

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.

3 participants