-
-
Notifications
You must be signed in to change notification settings - Fork 152
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
Make all hook params consistent #376
base: main
Are you sure you want to change the base?
Conversation
I would like to stay consistent. In one hook we now have So here it should either be |
Ah i know what you mean. i will check all hooks and push it in this PR. My goal is to be consistent too. |
Here is an overview with all Contao hooks: https://gist.github.com/Tastaturberuf/8851fc3348dcc229f4a49d81bc9220d0 On first sign i think we should suffix all models with e.g. |
@Tastaturberuf not sure you'd still be working on this - but: in Contao itself we usually use just Regarding the |
No description provided.