Closed ndianabasi closed 10 months ago
Since, this is going to be a big change (in terms of design). I will not be able to merge it right away.
Also, we have some plans around restructuring the hooks API, so I have to see how this change plays with that restructuring.
I will keep the PR on hold for now
Thanks for considering it.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Proposed changes
This PR injects the
ModelQueryBuilder
instance into the Modelafter
hooks.A use case for having the query instance available in the
after
hooks is for cleaning up changes to the model's class properties.Example:
Types of changes
What types of changes does your code introduce?
Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...