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

Update composer.json #1046

Open
wants to merge 1 commit into
base: 5.x
Choose a base branch
from

Conversation

arturodacostasoler
Copy link

@arturodacostasoler arturodacostasoler commented May 6, 2024

Included for laravel/framework and laravel/laravel to be able to try and test this package on Laravel 11.6.0:
|^11.0

Included for laravel/framework and laravel/laravel to be able to try and test this package on Laravel 11.6.0:
|^11.0
@ArlonAntonius
Copy link
Member

@arturodacostasoler Thank you for starting the effort on this, let us know if there's anything we can do to help update this package.

@arturodacostasoler
Copy link
Author

arturodacostasoler commented May 12, 2024

@arturodacostasoler Thank you for starting the effort on this, let us know if there's anything we can do to help update this package.

Hello and thank you very much for your reply.

It is necessary that the owner of this github package, please update and apply/commit the changes to the composer.json file, to include the two lines that allow using this package in Laravel 11, so that we can test it in that new Laravel version.

Mr. Mike Bronner also proposed making these same changes to the composer.json file in the following link, (also another file is modified in that link so the owner of this package must also evaluate those code line changes):

#1047

I'll be waiting for the composer.json file to be updated including the two lines of "|^11.0" to be able to test this package in Laravel 11, and perhaps also the additional changes proposed by Mr. Mike Bronner.

My best regards.

:)

@OliverWich
Copy link

OliverWich commented Jul 8, 2024

As Laravel 10 is slowly but surely moving towards its EOL, we wanted to update our application.

However, this dependency limitation to only support Laravel 9 and 10 of hyn/tenancy is blocking this transition as it is a very core package to us.

Is there any hope in this getting merged in the near future, or do we need to consider jumping ship to another library?

Support and development on this package seemed to have slowed down a lot in the last years, so this thought was already in the back of our minds.

@ArlonAntonius, what is the maintenance plan for hyn/multi-tenant, is there one or should we consider this as unmaintained/deprecated?
Deprecation would be a shame as this worked for us great for many years, but we need to move forward and cannot afford a core package limiting us with slow updates.

@arturodacostasoler
Copy link
Author

As Laravel 10 is slowly but surely moving towards its EOL, we wanted to update our application.

However, this dependency limitation to only support Laravel 9 and 10 of hyn/tenancy is blocking this transition as it is a very core package to us.

Is there any hope in this getting merged in the near future, or do we need to consider jumping ship to another library?

Support and development on this package seemed to have slowed down a lot in the last years, so this thought was already in the back of our minds.

@ArlonAntonius, what is the maintenance plan for hyn/multi-tenant, is there one or should we consider this as unmaintained/deprecated? Deprecation would be a shame as this worked for us great for many years, but we need to move forward and cannot afford a core package limiting us with slow updates.

I'm working with Laravel 11, so I had to use the following known Package as there was no update to our requests to simply update two files to be able to install this package on Laravel 11:

https://github.com/archtechx/tenancy

The archtechx/tenancy Package is interesting and updated on May 22 of 2024.

My best regards and is very sad not to have been able to Test this Package on Laravel 11.

:(

@ArlonAntonius
Copy link
Member

Heyhey everybody,

We strive to keep this package at least maintained by updating to later Laravel versions. Updating this package is slightly more difficult than our more recent packages due to it's slightly older approach.

I will try to update it this weekend, but it has many challenges 😅

The organization really recommends you to use https://github.com/tenancy/tenancy as it provides a more robust and modular setup compared to this package and is getting some amazing features in the future 😉

@OliverWich
Copy link

OliverWich commented Aug 2, 2024

The organization really recommends you to use https://github.com/tenancy/tenancy as it provides a more robust and modular setup compared to this package and is getting some amazing features in the future 😉

Is there a migration guide to switch to tenancy/tenancy?
As far as I've seen, there are some pretty big differences and some guide to safely change over without breaking existing deployments (e.g. database structures) when updating would be very much needed for a migration to be feasible.

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