Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork521
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
I went through all available messenger solutions in Laravel and I think that this project is the best pluggable solution out there. |
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 2 comments
-
hey@ronlinet, thanks for the discussion thread. I see a lot of value in the junction table. Since the thread can query all of the messages with the same id and We originally had foreign keys on the tables, but it introduced a lot of issues in some applications. Since you can override any of the models in the config, and the models could have different table names, this introduced conflicts. We decided to drop FK handling in the package directly and leave that up to the individual application to add or not. |
BetaWas this translation helpful?Give feedback.
All reactions
-
Hi@cmgmyr Thank you for considering the use case. After reviewing the things I do agree that a junction may be in this case an overkill. There is still an advantage of Sure things do work well in both cases :) . Feel free to close the issue. Thank you for your time. |
BetaWas this translation helpful?Give feedback.
