-
Notifications
You must be signed in to change notification settings - Fork 309
Conversation
Remove -s option for searchable model generation to fix compatibility with laravel 6.8
@babenkoivan please review this pull request. |
@lucasmichot can you please review this pull request because we are having issue after Laravel 6.8 update. |
@babenkoivan Any chance this could be reviewed ASAP as it's preventing people from upgrading to the latest versions of Laravel? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At some point, I guess Laravel should have a control on duplicate command shortucts
@babenkoivan I'm merging this PR since it's simple and causes no breaking changes. |
Merged and released on v4.0.2 |
Is it possible to have this backported to 3.x for those of us that can't upgrade to 4.x of the package because it's targeting ElasticSearch 7. We're doing some container hacks to override for now. |
Remove -s option for searchable model generation to fix compatibility with laravel 6.8
Issue : 305