Conversation
|
|
|
⚗️ Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch. To do so, temporarily add Shift's fork to the {
"repositories": [
{
"type": "vcs",
"url": "https://github.com/laravel-shift/slack-notification-channel.git"
}
]
}Then update your dependency constraint to reference this branch: {
"require": {
"beyondcode/slack-notification-channel": "dev-l11-compatibility",
}
}Finally, run: |
|
@mpociot do you need help with this? |
|
@mpociot would you mind taking a look at this and merging it please? |
|
If this library is being abandoned in favour of the official laravel library and you want to update to laravel 11 or later, you can reproduce the response functionality with a listener <?php
namespace App\Listeners;
use Illuminate\Notifications\Events\NotificationSent;
class SlackResponseListener
{
public function handle(NotificationSent $event): void
{
if ($event->channel !== 'slack') {
return;
}
if (! method_exists($event->notification, 'response')) {
return;
}
$event->notification->response($event->response);
}
}and then your notification gets the much more useful response class public function response(\Illuminate\Http\Client\Response $response): voidso you can get at the ts easier with |
This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:
l11-compatibilitybranchIf you do find an issue, please report it by commenting on this PR to help improve future automation.