-
-
Notifications
You must be signed in to change notification settings - Fork 714
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
Control logging when there is no changes #514
Conversation
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.
@freekmurze your opinion?
Hey, thanks for your effort! I've seen some codestyle and standard issues. But don't know if it's still simple enough for this package @freekmurze ? |
The StyleCI check still fails @akoSalman can you fix it? |
Apply fixes from StyleCI
When do you merge? I need this new feature😊 |
I think that I will have time tomorrow. Will comment the new release. Thanks so far! |
@Gummibeer Can you please merge this PR? I need to use it now in my project. Thanks. |
@akoSalman could you add this to the docs https://docs.spatie.be/laravel-activitylog/v3/advanced-usage/logging-model-events ? |
Sometimes you don't want to save logs that have no changes.
To use this feature you must set below properties on your model.