esnet / react-network-diagrams

Contains ESnet network mapping and circuit rendering code, as used in the ESnet portal.
http://software.es.net/react-network-diagrams
Other
196 stars 37 forks source link

PropTypes `Invalid argument supplied to oneOfType` on import #41

Closed rudolph9 closed 2 years ago

rudolph9 commented 5 years ago

A dump from my chrome console.

index.js:1 Warning: Invalid argument supplied to oneOfType. Expected an array of check functions, but received undefined at index 1.
console.<computed> @ index.js:1
printWarning @ factoryWithTypeCheckers.js:23
createUnionTypeChecker @ factoryWithTypeCheckers.js:372
../../node_modules/react-network-diagrams/lib/components/ConcatenatedCircuit.js @ ConcatenatedCircuit.js:387
__webpack_require__ @ bootstrap:788
fn @ bootstrap:149
../../node_modules/react-network-diagrams/lib/index.js @ index.js:20
__webpack_require__ @ bootstrap:788
fn @ bootstrap:149
./pages/Index.jsx @ module.js:22
__webpack_require__ @ bootstrap:788
fn @ bootstrap:149
./pages/index.js @ index.js:1
__webpack_require__ @ bootstrap:788
fn @ bootstrap:149
(anonymous) @ next-client-pages-loader.js:3
register @ page-loader.js:179
registerPage @ page-loader.js:221
register @ index.js:280
../../node_modules/next/dist/build/webpack/loaders/next-client-pages-loader.js?page=%2F&absolutePagePath=%2Fhome%2Fkrudolph%2FProjects%2Fjs-network-viewer%2Fapps%2Fnetwork-viewer%2Fpages%2Findex.js!./ @ next-client-pages-loader.js:2
__webpack_require__ @ bootstrap:788
fn @ bootstrap:149
0 @ index.js:3
__webpack_require__ @ bootstrap:788
checkDeferredModules @ bootstrap:45
webpackJsonpCallback @ bootstrap:32
(anonymous) @ index.js?ts=1566946336249:1

DeepinScreenshot_chromium_20190827161003

Note, this is running in a next.js dev server. A tar of an app that throws the warning is attached here react-network-diagrams-bug-proptypes-warning-on-import.tar.gz

/app » node --version
v11.15.0
rudolph9 commented 5 years ago

the entirety of the app is simply

import React from 'react';
import { TrafficMap } from "react-network-diagrams";

const App = () => {
  return (<div>Hello Wrold</div>)
}

export default App;
yusrilfirza commented 3 years ago

I have the same issue, can you please merge the pull request ?

sartaj10 commented 2 years ago

Deployed as part of v1.0.0 release