- Issue created by @mikkmiggur
- Merge request !6598Issue#3421304: Add logger message in config import. → (Open) created by sakthi_dev
- Status changed to Needs review
10 months ago 4:50pm 14 February 2024 This component is called 'config' and 'config_sync' for logging currently on the dev branch. It's unclear which to pick here.
- Status changed to Needs work
10 months ago 4:51pm 14 February 2024 - 🇪🇪Estonia hkirsman
I think we just missed that it would be better to inject it.
- 🇪🇪Estonia hkirsman
Hm, this then needs also update on Drush side if we're going for injection.
- 🇧🇪Belgium borisson_ Mechelen, 🇧🇪
We should follow the deprecation protocol we've had in multiple places, use DI, make it nullable and throw a deprecation (and instanciate the service) when the argument is null.
Are we sure we want to do this logging? It might be a lot of additional logs written during config imports?
- 🇪🇪Estonia hkirsman
Any link to some PR's or commits where that "deprecation protocol" was used?
It was the only way to log if and when block was deleted.