-
Notifications
You must be signed in to change notification settings - Fork 11.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
using Redis driver for queue, and problem with notifications. #29260
Comments
@samuhay Thanks for your reply BUT The version im currently on is 5.8 and you are sending me the 5.4 docs, which doesn't resolve my issue since the code works with sqs and sync drivers. |
I think it is because you have customized the Notification channel using receivesBroadcastNotificationsOn() in User model and you are listening event on App.User.* not on user.* So just remove receivesBroadcastNotificationsOn() from the User model and check again. Hope this helps Can you share code ? |
this is the function on user model that uses Notifiable trait. you are misunderstanding what im saying. if I have something misconfigured it would not work on sqs or sync driver aswell. |
Maybe related to #28701 ? |
@drievints I have tried couple more combinations, but there was no luck at all. it was showing the same behavior. and I moved on didn't change anything about it since I decided to give it a try for Redis once more. rebooted both Redis and ec2 than it started working, no idea why but I think there is some caching going on. I tried to dig deeper on framework and checked the files that are responsible for queueing but didn't see any caching so far. since you are Laravel maintainer would you please try to guess what would be causing the issue, I would love to read your comments. thank you. |
@igunultas did you read the issue I linked to? |
It's work for me though, I'm using laravel 5.8.27 |
I'm going to close this as a duplicate of #28701 I've already looked into it but currently stuck on figuring out where the messages get pulled back out from Redis. Any help is appreciated. |
Description:
While using Redis as a queue driver, and trying to send notification to broadcast it just ignores receivesBroadcastNotificationsOn on the model that has notifiable trait.
it keeps sending notifications to channel App.User.{userid}
instead of what receivesBroadcastNotificationsOn returns
if I use sync or sqs driver it just work fine.
Steps To Reproduce:
Create notification
Add receivesBroadcastNotificationsOn to model
set queue driver to Redis
The text was updated successfully, but these errors were encountered: