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

Is this bundle compatible (or when will be?) with Symfony 4? #297

Closed
Lifeformwp opened this issue Nov 22, 2017 · 14 comments
Closed

Is this bundle compatible (or when will be?) with Symfony 4? #297

Lifeformwp opened this issue Nov 22, 2017 · 14 comments

Comments

@Lifeformwp
Copy link

Hello, when I trying to install this bundle on my 4.0.0-RC1 composer gives me error that v3.4 is minimum-stability. Will you support ^4 and when? Or am I doing something wrong?

@bytehead
Copy link
Member

There is a PR #295, but the tests do still fail and it's not yet complete. But the goal is to support ^4.0

@Lifeformwp
Copy link
Author

Thank you!

@klodoma
Copy link

klodoma commented Dec 1, 2017

Any timeline when you will release the support for Symfony4?
As I see it's already developed in a branch. Thx.

@bytehead
Copy link
Member

bytehead commented Dec 4, 2017

@Lifeformwp @klodoma
There's a release/2.0.0-branch - could you test and give feedback if possible?

@klodoma
Copy link

klodoma commented Dec 5, 2017

@bytehead I will give it a try the next days as we plan to upgrade to Symfony4. I'll provide my feedback.

@bytehead
Copy link
Member

bytehead commented Dec 5, 2017

Thank you, I'll await your feedback!

@klodoma
Copy link

klodoma commented Dec 8, 2017

Hi, so my current feedback... , I am aborting the upgrade of our project to 4. There are too many dependencies in the project that are currently failing the upgrade.

Still, if I'll find some spare time I'll give it a shot and test your bundle on Symfony4.

@soifou
Copy link

soifou commented Dec 27, 2017

@bytehead I'm testing your release/2.0.0 branch and no problem to report so far 👍

I only had to create package and routing files in config/. You plan to create a contrib recipe for it, right?

@ErnadoO
Copy link
Contributor

ErnadoO commented Dec 28, 2017

Hello @bytehead

i'm testing release/2.0.0 too and i found one thing.

Like in your documentation, I use Orphanage like this:
$manager = $this->get('oneup_uploader.orphanage_manager')->get('products');

But Sf4 don't like it:

The "oneup_uploader.orphanage.products" service or alias has been removed or inlined when the container was compiled. You should either make it public, or stop using the container directly and use dependency injection instead.

I had fix it, i will send a PR soon ;)

@francoispluchino
Copy link

@bytehead For me, it's ok, but the PR #311 must be merged. When will the 2.0 version be released in stable?

@bytehead
Copy link
Member

bytehead commented Jan 8, 2018

@soifou Thanks for your feedback!
It is planned, but I don't know, when I have time for it. Wanna create a PR? 🙂

@francoispluchino #311 is merged. I think we should also provide a contrib receipe before release the stable 2.0.

@soifou
Copy link

soifou commented Jan 12, 2018

@bytehead PR done, waiting for 2.0 version to be released :)

@bytehead
Copy link
Member

Thank you! I'll release 2.0 asap!

@bytehead
Copy link
Member

Version 2.0.0 is out!
Thanks to everyone involved! 🎉

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

No branches or pull requests

6 participants