themsaid / wink

A Laravel-based publishing platform
MIT License
2.89k stars 378 forks source link

Laravel 11 compatiblity #391

Open phrane opened 6 months ago

phrane commented 6 months ago

Cannot install in Laravel Sail (11) environment. When running "sail composer require themsaid/wink", error is received with response, "Your requirements could not be resolved to an installable set of packages." Seems to be: PHP Version Compatibility: The error message indicates that your PHP version (8.3.3) does not meet the requirements specified by Wink. Wink requires PHP versions 7.1, 7.3, or higher (but is not accepting 8.3.3).

Laravel Framework Compatibility: Wink has specific requirements regarding the Laravel framework version. My root composer.json file specifies a requirement of Laravel version 11.0, which conflicts with the required versions by Wink.

Expected: Should be able to install wink using instruction on readme: composer require themsaid/wink php artisan wink:install php artisan storage:link

Actual: With L11 (Sail) the following error is received when attempting to require package:


Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - themsaid/wink[v0.0.1, ..., v0.2] require php ^7.1 -> your php version (8.3.3) does not satisfy that requirement.
    - themsaid/wink[v1.0, ..., v1.0.3] require php ^7.3 -> your php version (8.3.3) does not satisfy that requirement.
    - themsaid/wink[v1.1.0, ..., v1.2.0] require php ^7.3||^7.4 -> your php version (8.3.3) does not satisfy that requirement.
    - themsaid/wink[v1.2.1, ..., v1.3.1] require laravel/framework ^7.0||^8.0 -> found laravel/framework[v7.0.0, ..., v7.30.6, v8.0.0, ..., v8.83.27] but it conflicts with your root composer.json require (^11.0).
    - themsaid/wink v1.3.2 requires laravel/framework ^7.0||^8.0||^9.0 -> found laravel/framework[v7.0.0, ..., v7.30.6, v8.0.0, ..., v8.83.27, v9.0.0, ..., v9.52.16] but it conflicts with your root composer.json require (^11.0).
    - themsaid/wink v1.3.3 requires laravel/framework ^7.0||^8.0||^9.0|^10.0 -> found laravel/framework[v7.0.0, ..., v7.30.6, v8.0.0, ..., v8.83.27, v9.0.0, ..., v9.52.16, v10.0.0, ..., v10.48.2] but it conflicts with your root composer.json require (^11.0).
    - Root composer.json requires themsaid/wink * -> satisfiable by themsaid/wink[v0.0.1, ..., v0.2, v1.0, ..., v1.3.3].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.
You can also try re-running composer require with an explicit version constraint, e.g. "composer require themsaid/wink:*" to figure out if any version is installable, or "composer require themsaid/wink:^2.1" if you know which you need.```
tvbeek commented 6 months ago

As you can see there is already a PR #390 for this. You can test the branch from the PR and say if it works for you, that can help @themsaid to know if it works. πŸ˜ƒ

Franche020 commented 6 months ago

Hi, I was trying and I get this error:

PHP Fatal error: Class Wink\WinkAuthor contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Illuminate\Contracts\Auth\Authenticatable::getAuthPasswordName) in /var/www/html/vendor/themsaid/wink/src/WinkAuthor.php on line 23

Symfony\Component\ErrorHandler\Error\FatalError

Class Wink\WinkAuthor contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Illuminate\Contracts\Auth\Authenticatable::getAuthPasswordName)

at vendor/themsaid/wink/src/WinkAuthor.php:23 19β–• @property CarbonInterface $created_at 20β–• @property array|null $meta 21β–• @property-read Collection $posts 22β–• / ➜ 23β–• class WinkAuthor extends AbstractWinkModel implements Authenticatable 24β–• { 25β–• /* 26β–• The attributes that aren't mass assignable. 27β–• *