search

video-dev / clappr-rtmp-plugin

📺 RTMP support for Clappr player.
http://clappr.io
Apache License 2.0
105 stars 46 forks source link

one bug and one enhance #78

Closed hexray-newbee closed 7 years ago

hexray-newbee commented 7 years ago

fix: https://github.com/hxl-dy/clappr-rtmp-plugin/commit/324ffeaa3f2b78c90001e56a07d8455a012ed159 enhance: https://github.com/hxl-dy/clappr-rtmp-plugin/commit/d494b1d526f9e1cc99faa00940578e95d601d468

flavioribeiro commented 7 years ago

@hxl-dy why do we need to track dist/ folder?

hexray-newbee commented 7 years ago

hi, @flavioribeiro ! npm install will check .gitignore and copy to .npmignore, then dist/ won't be downloaded. when I import the lib and webpack to my project, dist/rtmp.js cannot found.

npm -v
4.0.3
flavioribeiro commented 7 years ago

ok thanks!