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

[BUG] Notifications package 2.2.0 includes path to travis #716

Closed
rvsia opened this issue Sep 1, 2020 · 1 comment · Fixed by #720
Closed

[BUG] Notifications package 2.2.0 includes path to travis #716

rvsia opened this issue Sep 1, 2020 · 1 comment · Fixed by #720
Assignees
Labels
bug Something isn't working

Comments

@rvsia
Copy link
Contributor

rvsia commented Sep 1, 2020

Describe the bug

Sources UI cannot run Notification 2.0.0

ERROR in ./node_modules/@redhat-cloud-services/frontend-components-notifications/cjs/NotificationPortal.js
Module not found: Error: Can't resolve 'home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Alert/Alert.js' in '/home/rvsianskz/insightsproject/sources-ui/node_modules/@redhat-cloud-services/frontend-components-notifications/cjs'
 @ ./node_modules/@redhat-cloud-services/frontend-components-notifications/cjs/NotificationPortal.js 1:517-650

bundled package:

@redhat-cloud-services/frontend-components-notifications/cjs/NotificationPortal.js

Alert_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Alert/Alert.js"),TextContent_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Text/TextContent.js"),Text_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Text/Text.js"),AlertActionCloseButton_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Alert/AlertActionCloseButton.js"),CloseIcon=_interopDefault(require("@patternfly/react-icons/dist/js/icons/close-icon")),Card_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Card/Card.js"),CardBody_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Card/CardBody.js"),Button_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Button/Button.js"),Pagination_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/components/Pagination/Pagination.js"),Level_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/layouts/Level/Level.js"),LevelItem_js=require("home/travis/build/RedHatInsights/frontend-components/node_modules/@patternfly/react-core/dist/js/layouts/Level/LevelItem.js"),

@karelhala

@karelhala
Copy link
Collaborator

Yeah we've changed the build and introduced this bug in #700 we've fixed that in #711 but that'll reguire manual releases as the auto releaser was not triggered because of not hitting all affected packages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants