Problem/Motivation

Using the state system means that configuration can't be exported to code, and can't be easily overridden on a per-hostname basis.

Steps to reproduce

Install and configure the module.
Export the site's configuration.

Expected behavior:
* The module's configuration is included in the export.

Actual behavior:
* The settings are not exported.

Proposed resolution

Change the module to store all settings in configuration.

Remaining tasks

Rewrite the code.

User interface changes

n/a

API changes

All code related to configuration would be removed.

Data model changes

n/a

Issue fork cloudflare_node_cc-3507723

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

damienmckenna created an issue. See original summary.

damienmckenna opened merge request !8

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review

We've been using something like this for a while; I rewrote our current code to remove the state system entirely.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

FYI we've been using MR 8 since February, 2025 on a fleet of 30 sites and it has been working well for us.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #3616312: Plan for Cloudflare Node CC 1.1.0

  • damienmckenna committed 46e07a5b on 1.x 
    Issue #3507723 by DamienMcKenna: Use configuration for module's...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

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.