Skip to content

Commit

Permalink
chore: Upgrade @hawtio/react to 1.0.7 (#340)
Browse files Browse the repository at this point in the history
* Pin victory-core resolution to 36.8.6 to remove warning about react/charts

* management-service.ts
 * Fix casting compilation error

* Suppress Critical Dependency expression warning
  • Loading branch information
phantomjinx committed Feb 23, 2024
1 parent 598229a commit a3a03e8
Show file tree
Hide file tree
Showing 13 changed files with 293 additions and 471 deletions.
2 changes: 1 addition & 1 deletion packages/kubernetes-api-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
},
"dependencies": {
"@hawtio/online-kubernetes-api": "workspace:*",
"@hawtio/react": "^1.0.3",
"@hawtio/react": "^1.0.7",
"@patternfly/react-core": "^4.278.1",
"@patternfly/react-styles": "^4.92.6",
"@patternfly/react-table": "^4.113.7",
Expand Down
4 changes: 4 additions & 0 deletions packages/kubernetes-api-app/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,10 +151,14 @@ module.exports = () => {
chunkFilename: 'static/js/[name].chunk.js',
assetModuleFilename: 'static/media/[name].[hash][ext]',
},

// For suppressing warnings that stop app running
ignoreWarnings: [
// For suppressing sourcemap warnings coming from some dependencies
/Failed to parse source map/,
/Critical dependency: the request of a dependency is an expression/,
],

resolve: {
extensions: ['.js', '.ts', '.tsx', '.jsx'],
alias: {
Expand Down
2 changes: 1 addition & 1 deletion packages/kubernetes-api/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
},
"dependencies": {
"@hawtio/online-oauth": "workspace:*",
"@hawtio/react": "^1.0.3",
"@hawtio/react": "^1.0.7",
"@types/jquery": "^3.5.29",
"@types/jsonpath": "^0.2.4",
"@types/node": "^18.19.17",
Expand Down
2 changes: 1 addition & 1 deletion packages/management-api-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
},
"dependencies": {
"@hawtio/online-management-api": "workspace:*",
"@hawtio/react": "^1.0.3",
"@hawtio/react": "^1.0.7",
"@patternfly/react-core": "^4.278.1",
"@patternfly/react-styles": "^4.92.6",
"@patternfly/react-table": "^4.113.7",
Expand Down
4 changes: 4 additions & 0 deletions packages/management-api-app/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -156,10 +156,14 @@ module.exports = () => {
chunkFilename: 'static/js/[name].chunk.js',
assetModuleFilename: 'static/media/[name].[hash][ext]',
},

// For suppressing warnings that stop app running
ignoreWarnings: [
// For suppressing sourcemap warnings coming from some dependencies
/Failed to parse source map/,
/Critical dependency: the request of a dependency is an expression/,
],

resolve: {
extensions: ['.js', '.ts', '.tsx', '.jsx'],
alias: {
Expand Down
2 changes: 1 addition & 1 deletion packages/management-api/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
},
"dependencies": {
"@hawtio/online-kubernetes-api": "workspace:*",
"@hawtio/react": "^1.0.3",
"@hawtio/react": "^1.0.7",
"eventemitter3": "^5.0.1",
"jolokia.js": "^2.0.0",
"jquery": "^3.7.0",
Expand Down
3 changes: 2 additions & 1 deletion packages/management-api/src/management-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -292,12 +292,13 @@ export class ManagementService extends EventEmitter {
const connNames: string[] = []
for (const container of containers) {
const url: URL = this.connectToUrl(pod, container)
const protocol = url.protocol as 'http' | 'https'
const connection: Connection = {
name: this.connectionKeyName(pod, container),
jolokiaUrl: url.toString(),

// Not necessary but included to satisfy rules of Connection object
scheme: url.protocol,
scheme: protocol,
host: url.hostname,
port: Number(url.port),
path: url.pathname,
Expand Down
2 changes: 1 addition & 1 deletion packages/oauth-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
},
"dependencies": {
"@hawtio/online-oauth": "workspace:*",
"@hawtio/react": "^1.0.3",
"@hawtio/react": "^1.0.7",
"@patternfly/react-core": "^4.278.1",
"@patternfly/react-styles": "^4.92.6",
"@patternfly/react-table": "^4.113.7",
Expand Down
4 changes: 4 additions & 0 deletions packages/oauth-app/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -152,10 +152,14 @@ module.exports = () => {
chunkFilename: 'static/js/[name].chunk.js',
assetModuleFilename: 'static/media/[name].[hash][ext]',
},

// For suppressing warnings that stop app running
ignoreWarnings: [
// For suppressing sourcemap warnings coming from some dependencies
/Failed to parse source map/,
/Critical dependency: the request of a dependency is an expression/,
],

resolve: {
extensions: ['.js', '.ts', '.tsx', '.jsx'],
alias: {
Expand Down
2 changes: 1 addition & 1 deletion packages/oauth/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
"prepack": "yarn build && yarn replace-version"
},
"dependencies": {
"@hawtio/react": "^1.0.3",
"@hawtio/react": "^1.0.7",
"@thumbmarkjs/thumbmarkjs": "^0.13.3",
"babel-jest": "^29.6.1",
"fetch-intercept": "^2.4.0",
Expand Down
2 changes: 1 addition & 1 deletion packages/online-shell/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"@fortawesome/react-fontawesome": "^0.2.0",
"@hawtio/online-kubernetes-api": "workspace:*",
"@hawtio/online-management-api": "workspace:*",
"@hawtio/react": "^1.0.3",
"@hawtio/react": "^1.0.7",
"@patternfly/react-core": "^4.278.1",
"@patternfly/react-styles": "^4.92.6",
"@patternfly/react-table": "^4.113.7",
Expand Down
4 changes: 4 additions & 0 deletions packages/online-shell/webpack.config.common.js
Original file line number Diff line number Diff line change
Expand Up @@ -105,10 +105,14 @@ const common = (mode, publicPath) => {
chunkFilename: 'static/js/[name].chunk.js',
assetModuleFilename: 'static/media/[name].[hash][ext]',
},

// For suppressing warnings that stop app running
ignoreWarnings: [
// For suppressing sourcemap warnings coming from some dependencies
/Failed to parse source map/,
/Critical dependency: the request of a dependency is an expression/,
],

resolve: {
extensions: ['.js', '.ts', '.tsx', '.jsx'],
alias: {
Expand Down
Loading

0 comments on commit a3a03e8

Please sign in to comment.