search

horihiro / esp8266-google-home-notifier

MIT License
103 stars 23 forks source link

ip 関数と play 関数についてのご相談 #5

Closed mkttanabe closed 6 years ago

mkttanabe commented 6 years ago

田辺と申します。6月に ESP32 への対応を完了されて以来、 esp8266-google-home-notifier を活用させて頂いております。 先日より二度に渡り貴ライブラリを題材に勤務先のブログにて 以下の記事を公開させて頂きました。宜しければお手すきの折にご笑覧下さい。 http://dsas.blog.klab.org/archives/2018-08/52297379.html http://dsas.blog.klab.org/archives/2018-08/52297568.html

さて、手元では fork したライブラリコードをプライベートにカスタマイズして 扱っているのですが、もし今後改訂のご予定があれば、 オリジナルの google-home-notifier に存在する ip 関数と play 関数 (public) を 貴ライブラリへ加味して頂けないでしょうか？ デバイスの IP アドレスを固定可能な環境では ip 関数は有用ですし、 また、MP3 データのキャスト再生が可能であればさらに使途が広がるのではないかと 想像しております。 ご多忙のところ恐れ入りますが、おってご検討頂ければ幸いです。

horihiro commented 6 years ago

既にフォークしてカスタマイズしているんですね！ 該当部分だけでもPRしていただければ、マージしますがいかがでしょうか。

mkttanabe commented 6 years ago

早々にお返事をありがとうございます。後ほどリクエストをお送りしますので、内容をご確認の上ご判断下さい。

horihiro commented 6 years ago

マージさせていただきました (fa7fd50)。 その後細かな修正をしているので、リリースまではしばしお待ちください。

mkttanabe commented 6 years ago

（早々にありがとうございました。後日のリリースを楽しみにお待ちします。

horihiro commented 6 years ago

昨日、リリースしました！