- Issue created by @vangelisp
- ππΊHungary nagy.balint
Hi!
It is good, but similarly to π Fix "Ignore logging on config import" feature Needs work , we should always consider that there are other subscribers, and this could be a more global change.
For example sending a mail about a no change could be useless as well. - π¬π§United Kingdom vangelisp Fife, Scotland
I see, then we can mark this as a duplicate and continue on the #3388293 issue ? I believe it needs to be a combination of those 2 issues (the conditional actions need to apply on whatever subscriber event takes place)
- ππΊHungary nagy.balint
I think it is better to keep them separate.
This option could be implemented similarly to "Ignore logging on config import" checkbox.
So we could have a checkbox in the settings like "Omit logs with no modifications", and then the filter method could be in the base class, and each subscriber could call it to filter the logs.
- First commit to issue fork.
- Merge request !4Issue #3394087 by VangelisP, Ressinel: Avoid recording entries that have no changes to the configuration β (Open) created by Ressinel
- last update
9 months ago 10 pass - Status changed to Needs review
9 months ago 3:44pm 4 March 2024 - πΊπ¦Ukraine Ressinel πΊπ¦ Rivne
Hey!
I made this option globally and created MR.
Please, review this and let me know if we need additional changes. -
nagy.balint β
committed 39a14d7a on 4.x
Issue #3394087 by ressinel, VangelisP: Avoid recording entries that have...
-
nagy.balint β
committed 39a14d7a on 4.x
- Status changed to Fixed
6 months ago 9:13pm 23 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.