jascdk / Nilan_Homeassistant

Use your Home Assistant to control and read values from your Nilan air vent system
54 stars 25 forks source link

add landing page instead of nothing if nilan isnt connected #15

Closed sveip closed 4 years ago

sveip commented 4 years ago

There seems to be some issue with the closing of the http request, not sure how that is handled today. Page might be slow to load, but should work.

jascdk commented 4 years ago

Skærmbillede 2020-02-06 kl  21 57 39

It works - but I don't quite understand the purpose of it - is it supposed to look like this?

sveip commented 4 years ago

Correct. It just gives the user some feedback that the unit actually works, earlier it was a blank page. You can off course make it much more fancy, but I dont really see the need for that.

tor. 6. feb. 2020 kl. 22:00 skrev jascdk notifications@github.com:

[image: Skærmbillede 2020-02-06 kl 21 57 39] https://user-images.githubusercontent.com/35434108/73978069-ddb0e300-492b-11ea-9fca-8a080f7f9c78.png

It works - but I don't quite understand the purpose of it - is it supposed to look like this?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jascdk/Nilan_Homeassistant/pull/15?email_source=notifications&email_token=ABLR4L23USBASEBXGIB5V3TRBR27TA5CNFSM4KRBECK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELAYNSY#issuecomment-583108299, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLR4LY7HWV6Q56EPKSBEDLRBR27TANCNFSM4KRBECKQ .

jascdk commented 4 years ago

But what are your thoughts on the wifimanager library made by tzapu here on GitHub? Im always talking about it, but thinks its pretty nifty piece of code - it makes the coding more complex, but it should be possible to fit inside this code - but I never got it to work.

Here are som screenshots on a test nodemcu running the DEV branch of that wifimanager library.

Gives a lot of info.

Skærmbillede 2020-02-06 kl  22 09 05 Skærmbillede 2020-02-06 kl  22 07 40 Skærmbillede 2020-02-06 kl  22 07 15

jascdk commented 4 years ago

Also supports a way to input Mqtt credentials - together with wifi credentials - that way the config file would be unnecessary. Just a thought - but I need some help to make it :D

sveip commented 4 years ago

Im sure it's possible as long as there is enough space on the flash, but I dont have the time to make something that involved Im afraid.

tor. 6. feb. 2020 kl. 22:17 skrev jascdk notifications@github.com:

Also supports a way to input Mqtt credentials - together with wifi credentials - that way the config file would be unnecessary. Just a thought

  • but I need some help to make it :D

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jascdk/Nilan_Homeassistant/pull/15?email_source=notifications&email_token=ABLR4L4TTYTZWZIP4U5JXULRBR467A5CNFSM4KRBECK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELA2BGQ#issuecomment-583114906, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLR4L7MKWSZUMS5LXPIKBLRBR467ANCNFSM4KRBECKQ .