search

delian / node-unifiapi

UniFi API ported to Node.JS
GNU General Public License v3.0
49 stars 19 forks source link

version without WebRTC? #12

Closed malle-pietje closed 6 years ago

malle-pietje commented 6 years ago

@delian As the maintainer of the API client class this was derived from, I'm glad to see an API client for Node.js being developed and maintained! For a specific project, I plan to use the node-unifiapi package but I won't be needing the WebRTC stuff. Is there a version without the WebRTC support or will it be stable enough when not using WebRTC?

malle-pietje commented 6 years ago

NVM, I figured it out.

william-levi commented 6 years ago

@malle-pietje how did you install it without WebRtc version ?

william-levi commented 6 years ago

@delian can you help me please ?

delian commented 6 years ago

What do you want to do? To have an installation without th e webrtc module?

On Sat, 14 Apr 2018 at 09:06, william-levi notifications@github.com wrote:

@delian https://github.com/delian can you help me please ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/delian/node-unifiapi/issues/12#issuecomment-381312095, or mute the thread https://github.com/notifications/unsubscribe-auth/AAG7ZnMLWL3RYTQCsbMvEV6GzYeCVXG3ks5toa3wgaJpZM4S7AHy .

--

Delian

william-levi commented 6 years ago

yeah , without webrtc .. sorry for the noob question :)

malle-pietje commented 6 years ago

Unfortunately, for my project, the client requested to completely switch to another library due to their concerns wrt the WebRTC issues... So having a version with the WebRTC completely left out would be nice.

delian commented 6 years ago

This is what I am going to do, basically to remove the dependency on webrtc and saying that if someone wants to have it, it has to be explicitly installed

Delian

On Tue, Apr 17, 2018 at 9:06 AM malle-pietje notifications@github.com wrote:

Unfortunately, for my project, the client requested to completely switch to another library due to their concerns wrt the WebRTC issues... So having a version with the WebRTC completely left out would be nice.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/delian/node-unifiapi/issues/12#issuecomment-381871847, or mute the thread https://github.com/notifications/unsubscribe-auth/AAG7ZnqsajdQPpK7Bs92fxK481oln2CUks5tpZRwgaJpZM4S7AHy .

malle-pietje commented 6 years ago

That would be perfect!

william-levi commented 6 years ago

perfect ! 👍 Thanks

delian commented 6 years ago

Done

Delian

On Tue, Apr 17, 2018 at 9:20 AM malle-pietje notifications@github.com wrote:

That would be perfect!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/delian/node-unifiapi/issues/12#issuecomment-381875253, or mute the thread https://github.com/notifications/unsubscribe-auth/AAG7Zj95etSQWfK32fuG-bVZnIjs91czks5tpZfPgaJpZM4S7AHy .

delian commented 6 years ago

Done

Delian

On Tue, Apr 17, 2018 at 9:25 AM Delian Delchev delian.delchev@gmail.com wrote:

Done

Delian

On Tue, Apr 17, 2018 at 9:20 AM malle-pietje notifications@github.com wrote:

That would be perfect!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/delian/node-unifiapi/issues/12#issuecomment-381875253, or mute the thread https://github.com/notifications/unsubscribe-auth/AAG7Zj95etSQWfK32fuG-bVZnIjs91czks5tpZfPgaJpZM4S7AHy .

william-levi commented 6 years ago

@malle-pietje are you using raspberry pi in your project ?