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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow usage with Symfony 4 #20

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

Conversation

PetrHeinz
Copy link

@PetrHeinz PetrHeinz commented Mar 21, 2019

It seems that this package is in fact compatible with Symfony 4 already, so it should be enoughto change its requirements in composer.json. I've created this little PR to add the support. It will be incompatible with Symfony 5 though due to changes in FormTypeExtensionInterface.

I've added some jobs to the Travis build to support my claims. Builds on Symfony >=4.2 fail due to deprecations - it succeeds on 4.0 and Symfony is pretty good with keeping backward compatibility. Jobs with hhvm and Docker fail even on current master (see build of bd2cd20 in my fork or restart the build yourself).

Are you still able to merge and release a new version (3.1.0, I'd suppose), or is this project no longer maintained? Can I help? Thanks 馃檪

@ghost
Copy link

ghost commented Apr 15, 2019

Author of this bundle is not active anymore. Different package, but same author.
https://symfony.com/blog/finding-a-new-home-for-ivoryckeditorbundle

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.

None yet

1 participant