Skip to content

[10.x] Add broadcastAs function at BroadcastNotificationCreated - #48136

Merged
taylorotwell merged 2 commits into
laravel:10.xfrom
raphaelcangucu:broadcastAsNotification
Aug 23, 2023
Merged

[10.x] Add broadcastAs function at BroadcastNotificationCreated#48136
taylorotwell merged 2 commits into
laravel:10.xfrom
raphaelcangucu:broadcastAsNotification

Conversation

@raphaelcangucu

Copy link
Copy Markdown

I add this function so it will get the broadcast event name as the same way it does for other events.

I needed this, because I wanted to specify the event name , and without this function, it will get only the class name.

@cosmastech

Copy link
Copy Markdown
Contributor

We added something like this to our application within the past month. Would love to see it become a part of the framework.

@raphaelcangucu

Copy link
Copy Markdown
Author

We added something like this to our application within the past month. Would love to see it become a part of the framework.

Nice, let's see if gets approved.

@paulojosegamba

paulojosegamba commented Aug 22, 2023

Copy link
Copy Markdown

Nice one, i needed something like this a few months ago
Had to do it myself hehe

@driesvints driesvints changed the title Add broadcastAs function at BroadcastNotificationCreated, so it will … Add broadcastAs function at BroadcastNotificationCreated Aug 22, 2023
@driesvints driesvints changed the title Add broadcastAs function at BroadcastNotificationCreated [10.x] Add broadcastAs function at BroadcastNotificationCreated Aug 22, 2023
@digitdash

Copy link
Copy Markdown

Would love to see this added!

@thiago-a-delgado

Copy link
Copy Markdown

Ow, it would help a lot to have this feature approved!

@driesvints

Copy link
Copy Markdown
Member

Going to send in a revert for this because it's a breaking change. We can re-consider this for 11.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants