AidynMakhataev / laravel-surveyjs

[Deprecated] Laravel wrapper for SurveyJS library and Editor
Other
79 stars 58 forks source link

Conflict in complete new laravelproject #43

Closed svenssontobias closed 3 years ago

svenssontobias commented 3 years ago

In a complete new Laravel project (created and fully updated 7feb 2021) After running composer require aidynmakhataev/laravelsurveyjs

Problem 1

Any advice on how to solve this?

marcosrocha85 commented 3 years ago

+1

marcosrocha85 commented 3 years ago

@svenssontobias Here's what I did.

  1. Clone this repository to your hard drive.
  2. Add laravel/helpers (composer require laravel/helpers) to aidynmakhataev/laravelsurveyjs.
  3. Update laravelsurveyjs composer.json from:
    "require": {
    "illuminate/support": "~5",
    "cviebrock/eloquent-sluggable": "^4.0"
    },
    "require-dev": {
    "phpunit/phpunit": "~6.0",
    "orchestra/testbench": "~3.0"
    },

    to:

    "require": {
    "illuminate/support": "^8",
    "cviebrock/eloquent-sluggable": "^8.0"
    },
    "require-dev": {
    "phpunit/phpunit": "^9.3.3",
    "orchestra/testbench": "^3.0"
    },
  4. In your project, add the key to composer.json.
    "repositories": [
    {
      "type": "path",
      "url": "../path-you-cloned-this-project"
    }
    ],
  5. Add the package manually to your project's composer.json require: "aidynmakhataev/laravelsurveyjs": "@dev",
  6. Do a composer update
marcosrocha85 commented 3 years ago

It's good to mention that this is a workaround. The good practice is to create a merge request updating the project and creating a new version compatible to Laravel 8 (and minor versions as well).

AidynMakhataev commented 3 years ago

https://github.com/AidynMakhataev/laravel-surveyjs/pull/48 pull request was merged, you can now upgrade to 1.0.0 version with support Laravel 8