switcher cache tags are incorrect

Figured this out. MR coming.

Please commit my suggestion https://git.drupalcode.org/project/environment_indicator/-/merge_request...

Adding patch

Tested with these two patches together, and now when I save the configuration forms at either: /admin/config/development/environment-indicator/current or /admin/config/development/environment-indicator the toolbar is updated.

LGTM

            "drupal/environment_indicator": {
                "Toolbar should get the list cache tag, not the individual cache tags (3425788)": "https://www.drupal.org/files/issues/2024-06-05/3425788-8.patch",
                "Cache tags don't make it through to the toolbar (3425499)":"https://www.drupal.org/files/issues/2024-06-05/3425499-8.patch"
            },

trackleft2 → changed the visibility of the branch 4.x to hidden.

IMHO, this issue should be the next priority for this module since it fixes a major usability issue for testing other merge requests. It removes the need to clear the cache on every update to the environment indicator configuration.

Looks good, moving the tag merging outside the original location allows the context to be preserved.

I've updated this after it was reviewed and tested by the community with an update from @chx
See the slack conversation here: https://drupal.slack.com/archives/C1BMUQ9U6/p1723064619282849?thread_ts=...

Looks good again!

Uploading static patch to match the merge request.

Nice, RTBC as well. I'd forgotten all about this!

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