Add config schema

Created on 17 March 2024, 10 months ago

Problem/Motivation

If I enable this module in a functional test, then phpunit complains:

There was 1 error:

1) Drupal\Tests\...
Drupal\Core\Config\Schema\SchemaIncompleteException: No schema for gtm.settings

Steps to reproduce

Proposed resolution

Add config/schema/gtm.schema.yml

Remaining tasks

User interface changes

None

API changes

None

Data model changes

None

πŸ› Bug report
Status

Needs review

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States benjifisher Boston area

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

Merge Requests

Comments & Activities

  • Issue created by @benjifisher
  • Merge request !9Add a config schema β†’ (Open) created by benjifisher
  • Issue was unassigned.
  • Status changed to Needs review 10 months ago
  • πŸ‡ΊπŸ‡ΈUnited States benjifisher Boston area

    Three of the settings seem like they should be boolean, but they are defined in gtm.settings.yml as 0 rather than false, so the schema declares them as integer. Since this module is minimally maintained, I assume we should simply adapt the schema file to the existing config instead of updating the config, which would require an update function for existing sites.

Production build 0.71.5 2024