laravelista / comments

Native comments for your Laravel application.
MIT License
744 stars 142 forks source link

unable to install in laravel 9 #189

Open Shubhamlmp opened 2 years ago

Shubhamlmp commented 2 years ago

Info from https://repo.packagist.org: #StandWithUkraine Using version ^4.5 for laravelista/comments ./composer.json has been updated Running composer update laravelista/comments Loading composer repositories with package information Updating dependencies Your requirements could not be resolved to an installable set of packages.

Problem 1

You can also try re-running composer require with an explicit version constraint, e.g. "composer require laravelista/comments:*" to figure out if any version is installable, or "composer require laravelista/comments:^2.1" if you know which you need.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

mabasic commented 2 years ago

I think that this package does not support Laravel 9 atm.

ghost commented 2 years ago

Same issue. Bummer

jeroenpost86 commented 2 years ago

Same issue here: Only one of these can be installed: illuminate/pagination[v8.0.0, ..., 8.x-dev], laravel/framework[v9.0.0-beta.1, ..., 9.x-dev]. laravel/framework replaces illuminate/pagination and thus cannot coexist with it.

A fix is fairly easy by replacing all instances of illuminate/pagination with the new package, but this will cause it to break in laravel versions < 9.

bondarslavik commented 2 years ago

at master we have "illuminate/database": "^9.0" but at v4.5.0 "illuminate/database": "^8.0"

can somebody update the version?

m7vm7v commented 2 years ago

At the time being this solved my issue, I do agree it'd be great if the latest version is bumped -

composer require laravelista/comments:dev-master

djordje47 commented 1 year ago

At the time being this solved my issue, I do agree it'd be great if the latest version is bumped -

composer require laravelista/comments:dev-master

This unfortunately doesn't work with laravel 10.

- laravelista/comments dev-master requires illuminate/database ^9.0 -> found illuminate/database[v9.0.0, ..., v9.52.9] but these were not loaded, likely because it conflicts with another require.

m7vm7v commented 1 year ago

@djordje47 have you looked at the https://github.com/laravelista/comments/pull/198