Problem/Motivation

When installing or upgrading to the decoupled theme, the logo path will be incorrect as openy_carnation.settings.yml currently points to the profile.

Steps to reproduce

  • Install the theme or update from an old version with the default logo.
  • Observe the header logo is broken.

Proposed resolution

  • Update the config file
  • Add an update hook to check existing config for profiles/contrib/openy/themes/openy_themes/openy_carnation to the proper path based on the module location

Issue fork openy_carnation-3265284

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

froboy created an issue. See original summary.

froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
commented
Issue summary: View changes

podarok made their first commit to this issue’s fork.

podarok opened merge request !11

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Active » Needs review

Please, review

froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
commented
Status: Needs review » Needs work

@podarok looks great, just a tiny issue with the indentations but otherwise good to go.

  • froboy committed 40b729c on 2.0.x authored by podarok 
    Issue #3265284: Logo path points to old location
froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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