Skip to content
This repository has been archived by the owner on Jan 29, 2019. It is now read-only.

Some additions #9

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Some additions #9

wants to merge 4 commits into from

Conversation

marecl
Copy link

@marecl marecl commented Apr 7, 2018

Moved strings to PROGMEM to free RAM
Reading config directly from file (using buffer caused data corruption)
Motor position is properly read (related to upper issue)
Added buttons (up, down, reset)
Added button description to README
Slight code formatting
Incremented version (1.3.3)

For me MQTT is working, seems to be stable. Buttons don't interfere with web control (but they are in loop so if you're holding button you cant connect to blinds).

I thought about adding more settings to the webpage because every time I need to change something I need to reset configuration.

Cheers!

Moved strings to progmem to free RAM
Added buttons (see readme)
Slightly formatted code
Reading config directly from file (prevents corruption)
currentState is properly read (related to method of reading)
@nidayand
Copy link
Owner

Sorry for the looong delay - I've been a bit busy with work. Thanks for the big update - I'll give it a test next week as I'm currently travelling

@lukaszwitkowski
Copy link

Works great only remove delay and works with normalnspeed

@Bennedetto
Copy link

I can confirm that your PR works like a charm!

Motor rolls faster when using buttons
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants