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

laravel 8 update? #260

Open
albiima opened this issue Sep 14, 2020 · 12 comments
Open

laravel 8 update? #260

albiima opened this issue Sep 14, 2020 · 12 comments

Comments

@albiima
Copy link

albiima commented Sep 14, 2020

Hello,
are you planning to provide a laravel 8 update?
thanks.

@wwright216
Copy link

Bump ;) Laravel 8 has added some amazing features and would love to upgrade! Is there an ETA on the update by chance?

@martinduefrandsen
Copy link

Any update on this?

@carlfredrikhero
Copy link

Is it confirmed that the current version does not work with Laravel 8?

@martinduefrandsen
Copy link

Is it confirmed that the current version does not work with Laravel 8?

There are dependencies that needs to be updated for this package to work with Laravel 8 as far as I know.

@mazraara
Copy link

mazraara commented Oct 8, 2020

Any updates on this ? really looking forward to the L8 version

@martinduefrandsen
Copy link

@jarektkaczyk any news?

@preeco-privacy
Copy link

We'd love to see and L8 compatible version too. Any chance to get this done? Need a sponsor? Give us a hint pls...

@pakogn
Copy link

pakogn commented Dec 6, 2020

Hi, All

just install it with composer require sofa/eloquence:dev-master now.

Had told to @jarektkaczyk via Twitter to kindly review this repo. And what a fortune to receive his attention.

Thanks @jarektkaczyk, works like a charm!

@gfernandez-me
Copy link

Thanks, Pakogn, hope he updates this repo

@pakogn
Copy link

pakogn commented Dec 19, 2020

Thanks, Pakogn, hope he updates this repo

Hope so too! I'm already using it in production and all ok, with L8

composer require sofa/eloquence:dev-master

@gfernandez-me
Copy link

gfernandez-me commented Jan 2, 2021

For me still not working

SQLSTATE[42000]: [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Must specify table to select from. (SQL: select count(*) as aggregate from [tb_city] where (select *) is null) {"userId":"29","exception":"[object] (Illuminate\\Database\\QueryException(code: 42000): SQLSTATE[42000]: [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Must specify table to select from. (SQL: select count(*) as aggregate from [tb_city] where (select *) is null) at /var/www/vpapi/vendor/laravel/framework/src/Illuminate/Database/Connection.php:678)

....

#6 /var/www/vpapi/vendor/sofa/eloquence-base/src/Query/Builder.php(104): Illuminate\\Database\\Query\\Builder->get()
#7 /var/www/vpapi/vendor/laravel/framework/src/Illuminate/Database/Query/Builder.php(2361): Sofa\\Eloquence\\Query\\Builder->runPaginationCountQuery()
#8 /var/www/vpapi/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Builder.php(732): Illuminate\\Database\\Query\\Builder->getCountForPagination()

The Mappable is not working

I'm using

$this->queryBuilder = City::query();
return $this->queryBuilder->paginate($limit);

@DeepakAryal
Copy link

Hi, All

just install it with composer require sofa/eloquence:dev-master now.

Had told to @jarektkaczyk via Twitter to kindly review this repo. And what a fortune to receive his attention.

Thanks @jarektkaczyk, works like a charm!

Hi, I used dev-master and got this issue.

Argument 1 passed to Optimus\Genie\Repository::Optimus\Bruno{closure}() must be an instance of Illuminate\Database\Eloquent\Builder, instance of Sofa\Eloquence\Query\Builder given

any idea how i can resolve this?

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

8 participants