-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Automation editor: Add trigger/condition/action from target #28031
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
Conversation
…omation-element-by-target
…omation-element-by-target
…omation-element-by-target
…omation-element-by-target
…omation-element-by-target
…omation-element-by-target
src/panels/config/automation/add-automation-element/ha-automation-add-from-target.ts
Outdated
Show resolved
Hide resolved
| const floor = | ||
| entries[ | ||
| `floor${TARGET_SEPARATOR}${this.areas[valueId]?.floor_id || ""}` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
meaning floor could be undefined right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes then it renders the "Other areas"
| // floor areas, unassigned areas | ||
| if (valueType === "floor") { | ||
| return this._renderAreas( | ||
| entries[`floor${TARGET_SEPARATOR}${valueId ?? ""}`].areas! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can valueId be undefined/null?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, for all pseudo views like, other areas, devices, services, helpers....
Proposed change
Type of change
Example configuration
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: