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.

Sign up for GitHub

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

Jump to bottom

Larevel 10 issue and update dependency #601

Open

code23-barna wants to merge 2 commits into MedicOneSystems:master from code23-barna:master

Contributor

code23-barna commented Oct 23, 2023

If we use the package with Laravel 10 it currently fails with the following error message:

Too few arguments to function Illuminate\Database\Query\Expression::getValue(), 0 passed in /path/to/project/vendor/mediconesystems/livewire-datatables/src/Http/Livewire/LivewireDatatable.php on line 417 and exactly 1 expected

The Laravel 10 upgrade guide states to update these calls here: https://laravel.com/docs/10.x/upgrade#database-expressions

The other change is just an update to the latest reedware/laravel-relation-joins as version 5 is released.

code23-barna added 2 commits

October 23, 2023 13:06


          update manual casting (https://laravel.com/docs/10.x/upgrade#database…

63072e2

…-expressions)


          add latest reedware/laravel-relation-joins to dependencies

74c0bed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet