-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Question wrt setup #6
Comments
Hey, Funny that you mention this, I was running into the same issue earlier this week while trying to get everything back up and running after moving to a different house. Thanks for the suggestion on using the DoubleResetDetector, it would be easier to use than the solution discussed in #4 If you get this working I'm more than interested in a patch or pull request 👍. It might take me a couple of weeks to merge it back as I would like to get my local setup up and running to test things out. And my Iskra AM550 is not making things easy for me 😂 |
Hi Daniël
I’m limited in my time, but you will get it when done😎
Best regards
Eric
…Sent from my mobile
On 16 May 2021, at 11:22, Daniel de Jong ***@***.***> wrote:
Hey,
Funny that you mention this, I was running into the same issue earlier this week while trying to get everything back up and running after moving to a different house.
Thanks for the suggestion on using the DoubleResetDetector, it would be easier to use than the solution discussed in #4
If you get this working I'm more than interested in a patch or pull request 👍. It might take me a couple of weeks to merge it back as I would like to get my local setup up and running to test things out. And my Iskra AM550 is not making things easy for me 😂
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
[ { ***@***.***": "http://schema.org", ***@***.***": "EmailMessage", "potentialAction": { ***@***.***": "ViewAction", "target": "#6 (comment)", "url": "#6 (comment)", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { ***@***.***": "Organization", "name": "GitHub", "url": "https://github.com" } } ]
|
Hi Daniël I attached an updated and tested version. I leave the merge to you. Basically I just made these changes:
One thing I observed is that I can't use OTA to update (maybe it never worked. I my projects I always enter OTA mode explicitly, not icw the normal code) Thanks for the great example. My Home Assistant is now successfully tracking my Power usage :-) Best Regards |
Hi Daniël Small Update: OTA works! I just needed a wired connection for my Laptop to see them. void setup() |
Hi Daniel
Very nice project and very well coded. I have a concern wrt configuring process of the board. First time works great.
According the code it re-enters AP mode when connect to the WiFi fails. Two real life issues here:
-When WiFi is down, the board ends up in AP mode. Won't reconnect once the WiFi is back.
-No way to fix or adjust MQTT settings when the WiFi works.
I think adding the DoubleResetDetector library to enter AP config mode when pressing reset twice is better. That's what I'm going to try. Interested in the diff?
Best regards and thx for the great work
Eric
The text was updated successfully, but these errors were encountered: