sqlboiler icon indicating copy to clipboard operation
sqlboiler copied to clipboard

Added hooks to templates for relationships.

Open mmitten opened this issue 5 months ago • 3 comments

My attempt to resolve #445

Added into the templates for relationships as I did not see any other place that made sense for how the hooks otherwise behaved. Happy to follow up if a something was missed.

mmitten avatar Feb 01 '24 18:02 mmitten

While I see this as a good change, I wonder if this may cause issues for those who are used to the current behaviour 🤔

stephenafamo avatar Mar 27 '24 12:03 stephenafamo

Now that you mention it, it could be a breaking change, as adding another hook trigger where it was previously not expected could lead to some unpleasant side effects.

I am not opposed to tabling this for a major version revision, as the use case I wrote it for has since been resolved with something else.

mmitten avatar Mar 27 '24 14:03 mmitten

I'll label this as breaking and may add it to an eventual v5 release.

stephenafamo avatar Mar 27 '24 18:03 stephenafamo