- Issue created by @scott_euser
- @scott_euser opened merge request.
- Status changed to Needs review
over 1 year ago 1:07pm 4 July 2023 - Issue was unassigned.
- π¬π§United Kingdom scott_euser
Also combined patch attached for #3372256 + #3372260 since these have a merge conflict in the update hook (since both require a new update hoo)
- Status changed to Needs work
over 1 year ago 11:14pm 6 July 2023 - πΊπΈUnited States aaronpinero
@scott_euser I am thinking that I might, while working on the other issue, split this into two configuration options so logging for each type of notification (content updates vs content review) can be enabled/disabled separately. Right now, the option makes it seem like this option is just for logging with content update notifications.
- Assigned to scott_euser
- π¬π§United Kingdom scott_euser
Makes sense to me given there are two config pages, will update the MR with this shortly.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 5:32am 7 July 2023 - π¬π§United Kingdom scott_euser
Updated the MR splitting into two configurations. Thanks!
-
aaronpinero β
committed eaada328 on 1.0.x authored by
scott_euser β
Issue #3372256: Add option to disable logging completely
-
aaronpinero β
committed eaada328 on 1.0.x authored by
scott_euser β
- Status changed to Fixed
over 1 year ago 8:26am 18 July 2023 - π¬π§United Kingdom scott_euser
Thanks! Changed to fixed since its committed.
Automatically closed - issue fixed for 2 weeks with no activity.