Problem/Motivation

The module has no schema defined for the config file. We should add one.

Steps to reproduce

Run the https://www.drupal.org/project/config_inspector checks on a site using simple klaro and see the errors.

Proposed resolution

Add the schema file to config/schema.

CommentFileSizeAuthor
#6 3222334-add-a-configschema-6.patch592 bytesfran seva
#3 3222334-add-a-configschema-3.patch592 bytesfran seva

Comments

jcnventura created an issue. See original summary.

fran seva’s picture

fran seva commented
Assigned: Unassigned » fran seva
fran seva’s picture

fran seva commented
Assigned: fran seva » Unassigned
Status: Active » Needs review
StatusFileSize
new 3222334-add-a-configschema-3.patch592 bytes

I have attached a patch with the schema configuration taking into account the enabled property that will be added as part of #3222370: Allow disable Simple Klaro in configuration

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Needs work
Related issues: +#3222370: Allow disable Simple Klaro in configuration

As per the requests in #3222370: Allow disable Simple Klaro in configuration, please change the enabled type from 'integer' to 'boolean'.

fran seva’s picture

fran seva commented
Assigned: Unassigned » fran seva
fran seva’s picture

fran seva commented
Assigned: fran seva » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3222334-add-a-configschema-6.patch592 bytes

I have attached a new patch based on the revision.

  • fran seva authored 1be346a on 1.0.x 
    Issue #3222334 by fran seva, jcnventura: Add a config/schema file
jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.