search

gpressutto5 / laravel-slack

:hash: Slack notification for Laravel as it should be. Easy, fast, simple and highly testable.
MIT License
288 stars 41 forks source link

As of L5.7 class SlackMessage is no longer available #13

Closed ralphmoran closed 5 years ago

ralphmoran commented 5 years ago

As of L5.7 the class Illuminate\Notifications\Messages\SlackMessage is no longer available. I suggest adding a note about this class is now a separated package that must be install.

Thanks

Screen Shot 2019-04-11 at 5 21 19 PM
ssenol commented 5 years ago

I installed laravel/slack-notification-channel package.

composer require laravel/slack-notification-channel

gpressutto5 commented 5 years ago

Thank you @ralphmoran! I will fix this now. Since laravel-slack doesn't work at all without the notification channel I think it should be added to this package's requirements.

gpressutto5 commented 5 years ago

If you're using Laravel 5.7 you don't need to require laravel/slack-notification-channel. if it's not working for you, just run composer update. They fixed this breaking change on laravel/framework#26727. Now I need to require laravel/slack-notification-channel@^2.0 for Laravel 5.8 support, but it will break <5.8 support. It would be great if we could specify that this package requires:

illuminate/support >= 5.5 < 5.7
OR
illuminate/support ~5.7.28 AND laravel/slack-notification-channel ^1.0
OR
illuminate/support ^5.8 AND laravel/slack-notification-channel ^2.0