Problem/Motivation

Be nice and declare our config in config/schema.

Proposed resolution

Do it.

Issue fork epp-3348759

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

geek-merlin created an issue. See original summary.

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: Unassigned » elber

I can work on it.

elber opened merge request !8

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: elber » Unassigned
Status: Active » Needs review

Hi I added the config and schema files please revise.

heddn’s picture

heddn
Primary language English
Location Illinois
commented

This is doing far more then just adding config schema. Can the scope be limited?

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Status: Needs review » Reviewed & tested by the community

Hi I thought that I needed add the form settings also, sorry.

I revised your schema file and it is good.

Schema file is correct .
Issue's scope was fixed.
Module is working
Moving to RTBC.

vitaliyb98 made their first commit to this issue’s fork.

vitaliyb98’s picture

vitaliyb98 commented

Also removed skipping schema from tests:

  /**
   * {@inheritdoc}
   */
  protected $strictConfigSchema = FALSE;

vitaliyb98’s picture

vitaliyb98 commented
Status: Reviewed & tested by the community » Fixed

Merged! Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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