search

adam-boduch / laravel-grid

Laravel Grid
MIT License
9 stars 5 forks source link

Laravel Support 8.9 #6

Open victorchiriac89 opened 4 years ago

victorchiriac89 commented 4 years ago

Hello, can you please update so it works with the latest Laravel 8.9? In case something will be wrong we will let you know what you can fix.

adam-boduch commented 4 years ago

The project should be compatible with Laravel 8.x

victorchiriac89 commented 4 years ago

Hi Adam, you need to adjust the composer to work for 8.9, we have the latest Laravel code and we cannot require it, please test as well and see.

adam-boduch commented 4 years ago

Have a look at the composer.json:

"require": {
        "php": ">=7.2.0",
        "nesbot/carbon": "^2.32.0",
        "illuminate/http": "5.8|6.*|7.*|8.*",
        "illuminate/support": "5.8|6.*|7.*|8.*",
        "illuminate/view": "5.8|6.*|7.*|8.*",
        "illuminate/pagination": "5.8|6.*|7.*|8.*",
        "illuminate/validation": "5.8|6.*|7.*|8.*",
        "laravelcollective/html": "5.8|6.*",
        "laravel/helpers": "^1.3"
    },

Shouldn't it be working with Laravel 8.x?

BTW: Version 8.9 is not released yet. I see that 8.4 is the newest one.

victorchiriac89 commented 4 years ago

"require": { "php": "^7.2.5", "fideloper/proxy": "^4.2", "fruitcake/laravel-cors": "^2.0", "guzzlehttp/guzzle": "^7.0.1", "laravel/framework": "^8.0", "laravel/jetstream": "^1.4", "laravel/sanctum": "^2.6", "laravel/tinker": "^2.0", "livewire/livewire": "^2.0", "maatwebsite/excel": "^3.1" },

victorchiriac89 commented 4 years ago

I just installed the 8 version from Laravel via composer and I was not able to composer require this module.