Add an update hook to add config

Created on 15 July 2024, 4 months ago
Updated 30 July 2024, 4 months ago

Problem/Motivation

Issue purge the logs(like watchdog) Fixed adds a config file with some settings for the module, but nothing is going to add this for existing sites.

This causes, at least a notice/warning in http_client_log_cron when it tries to get the settings.

Aside: I also note that if this module is following semver, then it should probably have bumped the minor version when adding this feature, and not the patch version because purge the logs(like watchdog) Fixed wasn't fixing a bug.

Steps to reproduce

Upgrade the module from a pre 1.1.1 release.

Proposed resolution

Introduce an update hook to add the config for existing site installs.

Remaining tasks

Agree that an update hook should be added, and add it.

User interface changes

None.

API changes

None.

Data model changes

None.

🐛 Bug report
Status

Fixed

Version

1.1

Component

Code

Created by

🇬🇧United Kingdom steven jones

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024