Skip to content

Conversation

@choppedpork
Copy link
Contributor

@choppedpork choppedpork commented May 9, 2022

Adds support for a very no-name curtain / blind controller (I opted for "Curtain Module" as that's what the packaging and unit itself say) as found on ebay. The device belongs to the clkg category but is very simple and supports only four of the instructions (w/o any read-only status DPS). Do let me know if I missed anything!

The translations file had a mix of tabs and spaces, my editor decided to clean that up via autoformat on save but also expanded a couple of objects across more lines, happy to revert that if it's a step too far 🙂

Thanks for a great integration!

edit: just noticed that the diff view of the translation file doesn't actually show the added entries amongst formatting changes, the entries are select_motor_reverse_mode and number_travel_time_configuration.

make-all added a commit that referenced this pull request May 22, 2022
Travel time translation was supplied for the initial config, but not for options.

PR #169
@make-all
Copy link
Owner

Merged

@make-all make-all closed this May 22, 2022
@make-all
Copy link
Owner

The final version renames the config to follow the existing convention of having the type of device last in the name rather than first. So any devices you have configured will need reconfiguring after the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants