search

cybercog / laravel-ownership

Laravel Ownership simplify management of Eloquent model's owner.
https://komarev.com/sources/laravel-ownership
MIT License
89 stars 16 forks source link

Add Laravel 8 support #22

Closed WilburJZHAO closed 3 years ago

WilburJZHAO commented 3 years ago

Add Laravel 8 support

antonkomarev commented 3 years ago

@WilburJZHAO thank you for the contribution! Could you add changes to .travis.yml as well? Here is example how I added PHP8 support to another one package: https://github.com/cybercog/laravel-love/commit/0a8820e732e6aefaefc0b970c1ea7c82b7ad4f37

antonkomarev commented 3 years ago

@WilburJZHAO sorry, I was too tired when I wrote that comment and gave a link on a wrong commit. There is no need to add PHP8 support to add Laravel 8 support. My bad.

WilburJZHAO commented 3 years ago

@antonkomarev All good, I just took out the PHP 8 support.

antonkomarev commented 3 years ago

I'm away from my computer, so cannot make a release. Will do it on the next week.

antonkomarev commented 3 years ago

@WilburJZHAO sorry for long wait time, but adding new Laravel versions required more than just adding them in composer.json: #23

Laravel Ownership v5.3.0 has been released with Laravel 6.x, 7.x, 8.x support and PHP8 support.

WilburJZHAO commented 3 years ago

@antonkomarev Thank you!