Skip to content
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

Add Initial Support for Cecotec Readywarm 8400 #2701

Merged
merged 10 commits into from
Jan 4, 2025

Conversation

stquinn
Copy link
Contributor

@stquinn stquinn commented Dec 27, 2024

Added Support for Cecotec Readywarm 8400 Bladeless Heater.

https://www.amazon.co.uk/Consumption-Bladeless-Connected-Oscillation-Security/dp/B08LNQJTT4

make-all added a commit to stquinn/tuya-local that referenced this pull request Dec 28, 2024
- naming tweaks
- use lower case cancel in timer to use translations
- use comfort instead of auto to use translations
- mark some duplicate hvac_modes hidden to ensure consistent behaviour
- remove redundant readonly marking on inherently read-only attribute

PR make-all#2701
make-all added a commit that referenced this pull request Dec 30, 2024
When a hidden mapping maps to the same dps_val as a normal mapping, the
normal mapping should be used.

But in the previous code, the hidden mapping could be returned from
_find_map_for_value then filtered out later (but only if the hidden
marking was on the map, not an embedded condition).

Instead, do the filtering inside _find_map_for_value, and improve the
logic so that hidden at the map level does not hide all the conditions
within, as that is used in some more complex conditional mappings.

Also take into account available attributes.

PR #2701
@stquinn stquinn force-pushed the feature/cecotec_readywarm_support branch from 9ed1b45 to e4f669a Compare December 30, 2024 13:34
@stquinn
Copy link
Contributor Author

stquinn commented Dec 30, 2024

@make-all Just seen your commit. Let me test with that as removing the "Off Preset" doesnt actually work as well as I would like. There are a piece of the UI that says null. I will test with your changes and see how that works.

Stephen and others added 3 commits January 4, 2025 15:09
- merge primary_entity and secondary_entities to a single list
- split product name ready for use in device info fields
@make-all make-all merged commit 68abb7e into make-all:main Jan 4, 2025
4 checks passed
make-all added a commit that referenced this pull request Jan 4, 2025
- minor tweak to follow HA naming convention for devices.
jlikeme added a commit to jlikeme/tuya-local that referenced this pull request Jan 7, 2025
* main: (1374 commits)
  Products cleanup: Q-S
  Silvercrest kettle: assume boil operation when warm dp is missing.
  Device config: remove unused local variable
  Device config: reduce log verbosity
  Bump tinytuya dependency to 1.16.0
  Products cleanup: N-P
  Products cleanup: L-M
  Products cleanup: J-K
  Gautone PG103: use events for alarm and power warnings
  Products cleanup: H-I
  ESP Fort smartplug: fix indentation
  Products cleanup: E-G
  Docs for PR make-all#2701
  Cecotec ReadyWarm 8400: apply new config format changes
  Revert Fan Only to be standard constraint rather than default and make OFF preset visible
  Add back 'Off' Preset Mode, remove DP1 'off' constraint  and set an initial default DP1 value before Preset contraints are evaluated
  Update Preset Modes to remove the off preset as per PR conversation
  Fix Cancel Timer Functionality
  Removed empty lines
  Added new line to end of file
  ...

# Conflicts:
#	custom_components/tuya_local/manifest.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants