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 support for Recteq RT700 Grill #2741

Merged
merged 6 commits into from
Feb 4, 2025
Merged

Conversation

smue86
Copy link
Contributor

@smue86 smue86 commented Jan 6, 2025

No description provided.

@@ -0,0 +1,42 @@
name: RecTeq RT-700 Grill
products:
- id: 5518137548fda419c2e
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to be a device id rather than product id.

dps:
- id: 1
type: boolean
mapping:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

name is required, was the removal of name: hvac_mode accidental?

- combined primary and secondary entities into a single list
- split product name into components
Other review changes:
- Remove branding from top level name
- add icon to climate entity

PR make-all#2741
In absense of an explanation, revert the removal of name to make the config valid again.

PR make-all#2741
@make-all make-all added the awaiting feedback Needs more information label Jan 26, 2025
Since there has been no reply, go to merge without a product id.

PR make-all#2741
@make-all make-all removed the awaiting feedback Needs more information label Feb 4, 2025
@make-all make-all merged commit c9d35e2 into make-all:main Feb 4, 2025
4 checks passed
make-all added a commit that referenced this pull request Feb 4, 2025
- combined primary and secondary entities into a single list
- split product name into components
Other review changes:
- Remove branding from top level name
- add icon to climate entity

PR #2741
make-all added a commit that referenced this pull request Feb 4, 2025
In absense of an explanation, revert the removal of name to make the config valid again.

PR #2741
make-all added a commit that referenced this pull request Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants