Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

correct path for di.xml #8443

Merged
merged 1 commit into from
Jan 3, 2021
Merged

correct path for di.xml #8443

merged 1 commit into from
Jan 3, 2021

Conversation

fooman
Copy link
Contributor

@fooman fooman commented Dec 23, 2020

Purpose of this pull request

This pull request fixes error
Unable to load theme by specified key: ''
with created admin theme on product edit page when following the devdocs guide.

Affected DevDocs pages

https://devdocs.magento.com/guides/v2.3/frontend-dev-guide/themes/admin_theme_apply.html#specify_di

Also see discussion here
https://magento.stackexchange.com/questions/302592/unable-to-load-theme-by-specified-key-magento-2-3-4

This seems to be the last related ticket
magento/magento2#26433
that once implemented may make etc/adminhtml/di.xml work (again?). However as of now the proposed changes here avoid creating an unusable admin theme.

@keharper keharper added 2.3.x Magento 2.3 related changes 2.4.x Magento 2.4.x related changes Technical Updates to the code or processes that alter the technical content of the doc labels Jan 3, 2021
@keharper keharper changed the base branch from master to small-changes January 3, 2021 03:44
@keharper keharper changed the base branch from small-changes to master January 3, 2021 03:44
@keharper keharper changed the base branch from master to small-changes January 3, 2021 03:49
@keharper keharper merged commit 26726b1 into magento:small-changes Jan 3, 2021
@ghost
Copy link

ghost commented Jan 3, 2021

Hi @fooman, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.3.x Magento 2.3 related changes 2.4.x Magento 2.4.x related changes Progress: done Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants