Deployment erases excluded-node-titles (as info is stored in config)

Created on 24 May 2018, over 6 years ago
Updated 11 January 2024, 10 months ago

Hello.

This module is storing in config the list of specific modules whose title is excluded. This can be a problem because this config is being deployed through CMI and it is possible to override these values by accident.

So, here I'm proposing to store this list in State API instead of config, so it won't be shared between environments, avoiding the problem described above.

🐛 Bug report
Status

Needs work

Version

1.0

Component

Code

Created by

🇪🇸Spain plopesc Valladolid

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇫🇷France ericdsd France

    Hi i also think node list are not meant to be stored in config.
    If it's useful to some users for migration, then we might eventually allow user to choose where to store it in module's settings.
    Don't you think we might make this issue move forward instead of keeping using it with a patch forever ?
    I'll be happy to help testing.

  • 🇵🇭Philippines dsdeiz

    Yeah, I've only rerolled the patch.

Production build 0.71.5 2024