Skip to content

Android IP Cam support#6488

Merged
robbiet480 merged 8 commits into
home-assistant:devfrom
pvizeli:pydroid-ipcam
Mar 10, 2017
Merged

Android IP Cam support#6488
robbiet480 merged 8 commits into
home-assistant:devfrom
pvizeli:pydroid-ipcam

Conversation

@pvizeli
Copy link
Copy Markdown
Member

@pvizeli pvizeli commented Mar 8, 2017

Description:

Original written by Robbie: #5614
I have port it to async, refactor the whole code and create a external library pydroid-ipcam.

android_ip_webcam:
  host: 192.168.1.10
android_ip_webcam:
 - host: 192.168.1.10
   scan_interval: 120
 - host: 192.168.1.12
   username: admin
   password: 123456
android_ip_webcam:
  host: 192.168.1.10
  auto_discovery: false
  switches:
   - focus
  sensors:
   - battery_level
  motion_sensor: true

If user exposed functionality or configuration variables are added/changed:

@mention-bot
Copy link
Copy Markdown

@pvizeli, thanks for your PR! By analyzing the history of the files in this pull request, we identified @balloob, @fabaff and @rmkraus to be potential reviewers.

@pvizeli pvizeli changed the title [WIP] Android IP Cam support Android IP Cam support Mar 9, 2017
@robbiet480 robbiet480 merged commit 8f2009a into home-assistant:dev Mar 10, 2017
@thundergreen
Copy link
Copy Markdown

Will it expose all features in standard configuration?

@pvizeli
Copy link
Copy Markdown
Member Author

pvizeli commented Mar 10, 2017

Only active features

@pvizeli
Copy link
Copy Markdown
Member Author

pvizeli commented Mar 10, 2017

Oh I see a bug. I'll fix it

@thundergreen
Copy link
Copy Markdown

thundergreen commented Mar 10, 2017 via email

@dellablue
Copy link
Copy Markdown

Would it be too much trouble to also support custom ports? I have 7 old cell phones each on different port, 192.168.1.40:8040, 192.168.1.41:8041, etc. and right now the component throws errors. Thanks!

@pvizeli
Copy link
Copy Markdown
Member Author

pvizeli commented Mar 10, 2017

Can you print error that I can fix it?

Do you have try with port: xy Option?

@dellablue
Copy link
Copy Markdown

dellablue commented Mar 10, 2017 via email

@pvizeli
Copy link
Copy Markdown
Member Author

pvizeli commented Mar 10, 2017

Please add your config

@dellablue
Copy link
Copy Markdown

dellablue commented Mar 10, 2017 via email

@pvizeli
Copy link
Copy Markdown
Member Author

pvizeli commented Mar 10, 2017

Wrong config. Use port

@pvizeli
Copy link
Copy Markdown
Member Author

pvizeli commented Mar 10, 2017

@dellablue
Copy link
Copy Markdown

dellablue commented Mar 10, 2017 via email

@home-assistant home-assistant locked and limited conversation to collaborators Mar 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants