Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restores icons in language editor #3755

Merged
merged 1 commit into from
May 11, 2020

Conversation

valadas
Copy link
Contributor

@valadas valadas commented May 11, 2020

Closes #3754

There was a breaking change with I believe raw-loader or file-loader
or svg-loader or some such, we already had 2 other similar PRs for
this same issue in other areas.

After that dependency update, I think in 9.5.0,
we now need to add .default at the end of svg imports.

This fixes the same issue in the language resources editor.

I did a search to find if we missed this in other spots and did not find
any other instance.

image

Also, as discussed in Slack, we would like to get this in 9.6.1-rc2 since we have so many contributors right now working on localization and this is a very small front-end only change.

Closes dnnsoftware#3754

There was a breaking change with I believe raw-loader or file-loader
or svg-loader or some such, we already had 2 other similar PRs for
this same issue in other areas.

After that dependency update, I think in 9.5.0,
we now need to add `.default` at the end of svg imports.

This fixes the same issue in the language resources editor.

I did a search to find if we missed this in other spots and did not find
any other instance.
@valadas valadas added this to the 9.6.1 milestone May 11, 2020
@bdukes bdukes merged commit ab531c0 into dnnsoftware:release/9.6.1 May 11, 2020
@valadas valadas deleted the issue3754 branch April 14, 2022 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants