Problem/Motivation

NotifyGroupNode is a custom replacement for the group content enabler plugin from gnode in Group core. We add additional configuration settings, but do not define a schema for those settings. This results in a fatal error inside a test that's trying to configure group_notify since core throws an Exception about missing schema for these config knobs.

Steps to reproduce

Proposed resolution

Define the schema appropriately.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork group_notify-3558646

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

dww created an issue. See original summary.

dww opened merge request !10

dww opened merge request !11

  • dww committed f9a8cdb9 on 2.0.x 
    task: #3558646: Define the schema for the NotifyGroupNode config...

  • dww committed a759962c on 8.x-1.x 
    task: #3558646: Define the schema for the NotifyGroupNode config...
dww’s picture

dww
we/he/they
commented
Status: Active » Fixed

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.

dww’s picture

dww
we/he/they
commented

Whoops, those commits should have said "fix" not "task". Bummer.

Oh well, at least this is fixed and #3558643: Add basic notification test coverage can proceed...

  • dww committed a51c6107 on 2.0.x 
    fix: #3558646: Update the schema for the NotifyGroupNode config settings...

dww closed merge request !10

dww closed merge request !11

Status: Fixed » Closed (fixed)

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