valtech / aem-easy-content-upgrade

AEM Easy Content Upgrade simplifies content migrations in AEM projects
Other
61 stars 27 forks source link

aecu.ui.content overwrites the configured weekly operations window #225

Closed kwin closed 1 year ago

kwin commented 1 year ago

Due to the filter rule https://github.com/valtech/aem-easy-content-upgrade/blob/891f68c7aabed3c8ad54c57fe749508f8bd1bc83/ui.content/src/main/content/META-INF/vault/filter.xml#L4 any customization to the weekly maintenance window is overwritten with each installation of the package

kwin commented 1 year ago

Also this should rather be located in /apps/settings (https://experienceleague.adobe.com/docs/experience-manager-65/administering/introduction/configurations.html?lang=en#configuration-resolution) as /conf is really supposed to be used by admins with the Web UI only. One can easily disable a task configured in /apps/settings/granite/operations/maintenance/granite_weekly in the UI (this will create an override in /conf/global/settings/.... which hides the task resource).

nhirrle commented 1 year ago

Thanks @kwin ! Also for the Pull Request to fix this.