Problem/Motivation

Add Drupal 11 support for the module.

Issue fork lb_theme_switcher-3562413

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

andrii.tiupa created an issue. See original summary.

andrii.tiupa opened merge request !4

froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
commented

@andrii.tiupa thanks for this. It looks like config.typed exists in D10 so I think we should be ok with just a minor bump, but have you tested on a D10 site? Just wondering how much we need to bump the version.

andrii.tiupa’s picture

andrii.tiupa commented

@froboy Indeed, in D10 we have
protected $typedConfigManager = NULL,
which is why the old approach worked without issues.

But in D11 it has changed to
protected TypedConfigManagerInterface $typedConfigManager,
so it now throws a fatal error on D10. Because of that, we should bump the version.

froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
commented
Version: 1.1.0 » 2.0.0
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.

Status: Fixed » Closed (fixed)

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