paviro / MMM-FRITZ-Box-Callmonitor

This an extension for the MagicMirror. It provides a callmonitor for FRITZ!Box users alerting them about incoming calls.
29 stars 24 forks source link

Update of FRITZ!Box access #18

Closed Jopyth closed 8 years ago

Jopyth commented 8 years ago

Unfortuantely the python library can not be installed through the existing npm install instead I added another optional installation step for this.

Changes for #17 :

Other Changes:

paviro commented 8 years ago

Is there now way to do it without python? Does it create a xml file or is the python script directly parsed by node via console output?

use header appendix Isn't this only available in your fork?

Am 17.09.2016 um 15:54 schrieb Jopyth notifications@github.com:

use header appendix

Jopyth commented 8 years ago

No, without python it is no longer possible. However the general python environment should usually exist on the RPi. The old solution had an expire date, therefore I did not think it was useful to keep as an alternative. Or you would have to reboot the FRITZ!Box every now and then to make it work again.

It creates a .xml but mostly due to potentially multiple files (the call list, and multiple phone books might exist) and before I create some intermediate format, which stores multiple files, I thought real files might be better. But maybe the direct feed from console output is better.

Header appendix is reverted, it does not exist on this branch.

paviro commented 8 years ago

Yeah I know use it in the facial recognition stuff as well just like to keep the dependencies low :) but I am willing to accept it anyway.

In regards of the file, I am not really a fan of that... Dint you think there is a way to just send it over via console output like I do it in the facial recognition stuff without a separate file?

Am 17.09.2016 um 16:27 schrieb Jopyth notifications@github.com:

It

Jopyth commented 8 years ago

Definitely possible, yes.

Jopyth commented 8 years ago

I will change it to console output.

paviro commented 8 years ago

Perfect! Thank you for your hard work! :)

Am 17.09.2016 um 17:44 schrieb Jopyth notifications@github.com:

I will change it to console output.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

Jopyth commented 8 years ago

I think it should be finished now. Using stdout to transfer data + squashed some of my commits.

paviro commented 8 years ago

Merged thanks!