Fix issue with onReady callback and tweak webpack

.npmignore

@@ -0,0 +1,41 @@
+*.log
+npm-debug.log*
+yarn-error.log
+
+# Coverage directory used by tools like istanbul
+coverage
+.nyc_output
+
+# Dependency directories
+node_modules
+
+# npm package lock
+package-lock.json
+yarn.lock
+
+# project files
+src
+test
+examples
+CHANGELOG.md
+.travis.yml
+.editorconfig
+.eslintignore
+.eslintrc
+.babelrc
+.gitignore
+
+
+# lock files
+package-lock.json
+yarn.lock
+
+# others
+.DS_Store
+.idea
+
+.out
+.github
+.storybook
+.config
+stories

README.md

@@ -33,10 +33,13 @@ import ReactTwitchEmbedVideo from "react-twitch-embed-video"
 - Lowest version of React supported is now 16.8 (version React Hooks was introduced in)
 - Removed onUserLogin, it appears Twitch stopped supporting this all together based off the docs
 - Added targetId property for defining mutiple video players
+- `onPlayerReady` renamed to `onReady`
+- `onVideoPlay` renamed to `onPlay`
 
 ### Breaking Changes
 In version one I was mistakenly using `targetClass` prop for populating the `id` attribute for the Twitch Player. So I created a new prop called `targetId` to use to define more then one Twitch Player on your page. `targetClass` now properly sets the class for the player for proper styling.
 If you were one of the users the was creating more then on Twitch player on your page, just switch from using `targetClass` to `targetId` and things should still work properly.
+If you're using `onPlayerReady` or `onVideoPlay`, make sure to rename them to the new prop names.
 
 ## Development
 

index.js

@@ -1,2 +1 @@
-require('./dist/manifest');
 module.exports = require('./dist/index').default;

package.json

@@ -25,7 +25,8 @@
     "window-or-global": "^1.0.1"
   },
   "peerDependencies": {
-    "react": "^16.8.0"
+    "react": ">= 16.3.0",
+    "react-dom": ">= 16.3.0"
   },
   "devDependencies": {
     "@arkweid/lefthook": "^0.7.0",
@@ -51,6 +52,8 @@
     "eslint": "^4.9.0",
     "eslint-plugin-react": "^7.5.1",
     "eslint-plugin-react-hooks": "^2.2.0",
+    "react": "^16.13.0",
+    "react-dom": "^16.13.0",
     "react-hot-loader": "^4.5.3",
     "style-loader": "^0.19.0",
     "webpack": "^4.0.0",

src/usePlayerReady.js

@@ -26,14 +26,14 @@ const usePlayerReady = (EmbedApi, options) =>
       return;
     }
 
-    const { getPlayer } = EmbedApi;
     const { autoplay: isAutoPlay, muted: isMuted, onReady } = options || {};
-    const player = getPlayer();
+
+    const player = EmbedApi.getPlayer();
 
     ensureVolume(player, isMuted);
     ensureAutoPlay(player, isAutoPlay);
 
-    onReady && onReady(player);
+    onReady && onReady();
   }, [EmbedApi, options]);
 
 export { usePlayerReady };

stories/index.js

@@ -65,12 +65,12 @@ storiesOf("React Twitch Embed Video", module)
   .add("with video play callback", () => (
     <TwitchEmbedVideo
       channel="talk2megooseman"
-      onVideoPlay={action("onVideoPlay")}
+      onPlay={action("onVideoPlay")}
     />
   ))
   .add("with player ready callback", () => (
     <TwitchEmbedVideo
       channel="talk2megooseman"
-      onPlayerReady={ignoreArgsAction("onPlayerReady")}
+      onReady={ignoreArgsAction("onPlayerReady")}
     />
   ));

webpack.prod.config.babel.js

@@ -45,12 +45,9 @@ module.exports = {
         ]
     },
 
-    optimization: {
-        runtimeChunk: { name: 'manifest' }
-    },
-
     plugins: [
         // Clean dist folder
         new CleanWebpackPlugin(['dist/*.*']),
     ]
+
 };