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

Updating frontend-component libs and add stripAllPfStyles flag to fec.config.js #10

Merged

Conversation

ammont82
Copy link
Collaborator

Try to solve CSS conflicts in the OCM UI. We have a solution which is not ideal but will unblock you. The problem is that the assisted installer app, depends on the assisted installer UI and depends on the openshift plugin SDK. Which still requires pf4.
This makes a real mess with dependency deduplication. It means the assisted installer UI has several instances of PF installed within node modules, which makes a kind of a mess in certain webpack config that is tasked with removing PF styles dependencies from HCC UI modules.

To solve this @Hyperkid123 propose this temporary solution: RedHatInsights/frontend-components#2019

Copy link

openshift-ci bot commented Jun 20, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ammont82

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 20, 2024
@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jun 20, 2024
@ammont82 ammont82 merged commit 0ef6900 into openshift-assisted:main Jun 20, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant