-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add support for relationships #17
Conversation
…el->relationship())`)
…del->relationship())`)
… in a real-world application it seems more expressive by adding it on constructor-level rather than on method-level
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, thanks for the great PR! Well documented and adds a lot of value 👌 I've added a couple questions in the PR review but apart from that, this looks good to merge
…tatsWriter to StatsQuery
…g this behaviour now via public "groupByWeek()" functionality
@AlexVanderbist thanks for reviewing 👍 In my opinion we're good to go. After looking at the existing code I saw that What do you think? |
Perfect, LGTM! I'll merge, make |
Thanks for creating this package 👍
I'm adding this realtionship-support because I'm in need of this in a tenant-based stats-approach. The following discussion #5 was very useful for it. After some initial thoughts on how to achieve this I remembered how easy it is with
spatie/laravel-query-bullder
. I took this as an inspiration.Feel free to follow up on the commit messages which may explain some changes.