Contrib.social

[Follow-up] action.configuration.user_remove_role_action could also use a UserHasRole constaint

Open on Drupal.org →
Created on 22 February 2025, about 1 month ago

Problem/Motivation

In the parent issue we validate a Role Eixsts. this could be more specific in this specific case where we check if the user has the role. Although im not sure that needs to be an error, since you might just want to remove a role from all users.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

✨ Feature request
Status

Active

Version

11.0 🔥

Component

configuration system

Created by

🇳🇱Netherlands bbrala Netherlands

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

Comments & Activities

  • Issue created by @bbrala
  • Comment 13 days ago →
    🇧🇪Belgium borisson_ Mechelen, 🇧🇪

    The parent issue was committed.

    Since it's about removing a role, I'm not sure if we need to have strict validation.
    However it could lead to an action that doesn't do anything when it has a role configured that does not exist I guess.

  • Comment 12 days ago →
    🇳🇱Netherlands bbrala Netherlands

    I kinda agree that a noop might not really be a problem.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024