-
Notifications
You must be signed in to change notification settings - Fork 510
Add logs for change of registration form settings #6544
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 logs for change of registration form settings #6544
Conversation
|
Maybe we can use this opportunity to add proper diff logging to registration form setting changes? I don't like the idea of hardcoding a single case here and not logging anything else... So basically, do something similar to what we do in |
Co-authored-by: Adrian <[email protected]>
Also use the regform's log method which links the log record to the regform (it allows filtering on the log view page)
b10b0b1 to
ced43e3
Compare
|
Yes, it's consistent with the log messages for creation. The text there is the module, not the log subject (that's a mistake in my commit msg ;)). And if you view one of those messages, there is a button near the bottom of the modal to filter all log messages for that regform - so you see updates, creation, etc |

Request to add logs to action 'change of registration form settings'.