Closed logonoff closed 6 days ago
@logonoff: This pull request references Jira Issue OCPBUGS-44578, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @yapei
The bug has been updated to refer to the pull request using the external bug tracker.
/retest
@logonoff: This pull request references Jira Issue OCPBUGS-44578, which is valid.
Requesting review from QA contact: /cc @yapei
/retest-required
Remaining retests: 0 against base HEAD 2705900f00bffab1e284d16d1dd8e4bdf5b0b5b7 and 2 for PR HEAD 546e6d632c46e8e5db217aa25fb52b039aa51527 in total
@logonoff: This pull request references Jira Issue OCPBUGS-44578, which is valid.
Requesting review from QA contact: /cc @yapei
@logonoff: all tests passed!
Full PR test history. Your PR dashboard.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: logonoff, lokanandaprabhu, spadgett
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@logonoff: Jira Issue OCPBUGS-44578: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-44578 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
Distgit: openshift-enterprise-console This PR has been included in build openshift-enterprise-console-container-v4.18.0-202411152008.p0.g3953d93.assembly.stream.el9. All builds following this will include this PR.
Fixes:
Fixes: https://issues.redhat.com/browse/OCPBUGS-44578
Analysis / Root cause:
When importing
svg
/png
files as anasset/resource
, the url is exported ESM-style, while the typescript we have defined sets it as a CommonJS one.This problem dosen't seem to exist with the
file-loader
we currently have, but it still works, so this PR helps split up my webpack 5 PRSolution Description:
Rewrite CJS imported png/jpg/jpeg/svg/... to ESM imports
Screen shots / Gifs for design review:
n/a
Unit test coverage report:
n/a
Test setup:
Nothing should change, icons should look the exact same
Browser conformance: