Update Satel Integra #37609
Update Satel Integra #37609
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
📝 WalkthroughWalkthroughThe documentation for the Satel Integra integration has undergone a complete overhaul. The configuration section now references a configuration flow instead of requiring manual YAML entries. Detailed options such as code, partitions, zones, outputs, and switchable outputs have been removed, leaving only a simplified configuration for Host and Port. A new "Settings" section has been added to explain how to manage these settings through the Home Assistant interface. Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant HA as Home Assistant UI
participant CF as Configuration Flow
U->>HA: Initiates Satel Integra integration setup
HA->>CF: Requests configuration form
CF->>U: Displays simplified form (Host, Port)
U->>CF: Submits configuration values
CF->>HA: Updates integration settings via UI
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (3)
source/_integrations/satel_integra.markdown (3)
64-64: Grammar Correction – Possessive Pronoun
The phrase “Each partition will have it's own alarm panel…” contains an incorrect contraction. It should be “its own alarm panel.” Please update to maintain professional documentation standards.🧰 Tools
🪛 LanguageTool
[uncategorized] ~64-~64: Did you mean “its” (the possessive pronoun)?
Context: ...option forms. Each partition will have it's own alarm panel, each zone and output w...(ITS_PREMIUM)
74-74: Punctuation and Style Improvement
In the sentence “Afterwards fill in the required fields for each entry type…”, consider adding a comma after "Afterwards" or switching to “Afterward” (as preferred in American English) to improve flow and consistency with style guidelines.
75-75: Clarification on Deletion Behavior
The sentence on line 75 could be rephrased for clarity. The current wording—“Deleting an entry removes the corresponding entry from the configuration, but the entity will still exist, you will manually have to remove this from the list afterwards.”—might be interpreted as a single run-on sentence. Consider splitting it or adding a semicolon for clarity. For example:“Deleting an entry removes it from the configuration; however, the entity remains and must be manually removed from the list afterward.”
🧰 Tools
🪛 LanguageTool
[locale-violation] ~75-~75: In American English, ‘afterward’ is the preferred variant. ‘Afterwards’ is more commonly used in British English and other dialects.
Context: ...ually have to remove this from the list afterwards. Having configured the zones and the o...(AFTERWARDS_US)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
source/_integrations/satel_integra.markdown(2 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_integrations/satel_integra.markdown
[uncategorized] ~64-~64: Did you mean “its” (the possessive pronoun)?
Context: ...option forms. Each partition will have it's own alarm panel, each zone and output w...
(ITS_PREMIUM)
[formatting] ~73-~73: Consider adding a comma after ‘Afterwards’ for more clarity.
Context: .../eba00284-c9fb-4678-9f46-1e67bd806ab3) Afterwards fill in the required fields for each en...
(CONJUNCTIVE_LINKING_ADVERB_COMMA_PREMIUM)
[locale-violation] ~75-~75: In American English, ‘afterward’ is the preferred variant. ‘Afterwards’ is more commonly used in British English and other dialects.
Context: ...ually have to remove this from the list afterwards. Having configured the zones and the o...
(AFTERWARDS_US)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - home-assistant-docs
- GitHub Check: Header rules - home-assistant-docs
- GitHub Check: Pages changed - home-assistant-docs
🔇 Additional comments (3)
source/_integrations/satel_integra.markdown (3)
45-46: Config Flow Inclusion Check
The inclusion of{% include integrations/config_flow.md %}appears correctly placed to signal the switch from YAML configuration to a configuration flow. Please verify that the referenced file accurately reflects the new integration configuration flow.
56-59: Addition of the Settings Section
The new "## Settings" section clearly explains that partitions, zones, and outputs are now managed via the Home Assistant UI options. This aligns with the PR objective to simplify configuration. No changes are needed unless further clarification is desired in terms of UI navigation.
60-63: Partitions, Zones and Outputs Overview
The "### Partitions, Zones and Outputs" heading and accompanying description offer a clear overview of the new management flow using option forms. The content is informative and aligns with the integration’s updated interaction model.
| {% configuration_basic %} | ||
| Host: | ||
| description: The IP address of the Satel Integra ETHM module. | ||
| Port: | ||
| description: The port on which the ETHM module listens for clients using integration protocol. | ||
| required: false | ||
| default: 7094 | ||
| type: integer | ||
| code: | ||
| description: User password, it's needed for making use of the switchable_outputs. It's recommended not to use admin password. | ||
| required: false | ||
| type: string | ||
| partitions: | ||
| description: List of the partitions to operate on. | ||
| required: false | ||
| type: [integer, list] | ||
| keys: | ||
| name: | ||
| description: Name of the partition. | ||
| required: true | ||
| type: string | ||
| arm_home_mode: | ||
| description: The mode in which the partition is armed when 'arm home' is used. Possible options are `1`,`2` or `3`. For more information on what the differences are between them, please refer to Satel Integra manual. | ||
| required: false | ||
| default: 1 | ||
| type: integer | ||
| zones: | ||
| description: "This parameter lists the zones (or inputs) that will be visible by Home Assistant. For each zone, a proper ID must be given as well as its name. The name is arbitrary and does not need to match the one specified in Satel Integra alarm configuration." | ||
| required: false | ||
| type: [integer, list] | ||
| keys: | ||
| name: | ||
| description: Name of the zone. | ||
| required: true | ||
| type: string | ||
| type: | ||
| description: The zone type. | ||
| required: false | ||
| default: motion | ||
| type: string | ||
| outputs: | ||
| description: "Very similar to zones, but with outputs. Satel Integra uses outputs to inform external systems about different events. For example power failure, or that alarm started counting for exit or some other user-defined condition. They may be used for simple alarm-based automation. For more information please refer to Satel homepage and forums." | ||
| required: false | ||
| type: [integer, list] | ||
| keys: | ||
| name: | ||
| description: Name of the output. | ||
| required: true | ||
| type: string | ||
| type: | ||
| description: The type of the device - just for presentation. | ||
| required: false | ||
| default: motion | ||
| type: string | ||
| switchable_outputs: | ||
| description: "Switchable outputs. These will show up as switches within Home Assistant." | ||
| required: false | ||
| type: [integer, list] | ||
| keys: | ||
| name: | ||
| description: Name of the output. | ||
| required: true | ||
| type: string | ||
| {% endconfiguration %} | ||
|
|
||
| ## Full examples | ||
| Code: | ||
| description: Optional code that will be used to toggle switchable outputs. | ||
| {% endconfiguration_basic %} |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Review the Simplified Configuration Block
The new {% configuration_basic %} block lists three settings: Host, Port, and Code. Note that the PR objectives and AI summary indicate a move to a simplified configuration focusing on Host and Port only. If the “Code” field is no longer required, consider removing it to align with the documentation objectives. Otherwise, update the documentation to clarify its purpose.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
source/_integrations/satel_integra.markdown (3)
60-62: Typographical Correction: Replace “it's” with “its”
In line 62, the phrase “Each partition will have it's own alarm panel…” uses an incorrect possessive form. Changing “it's” to “its” will improve the clarity and correctness of the text.🧰 Tools
🪛 LanguageTool
[uncategorized] ~62-~62: Did you mean “its” (the possessive pronoun)?
Context: ...option forms. Each partition will have it's own alarm panel, each zone and output w...(ITS_PREMIUM)
72-72: Punctuation Suggestion for Improved Readability
In line 72, consider adding a comma after “Afterwards” in “Afterwards fill in the required fields for each entry type” to enhance readability.
73-73: Locale Consistency: Review Usage of “afterwards”
In line 73, the word “afterwards” is used. If the documentation’s style guide prefers American English, consider using “afterward” for consistency.🧰 Tools
🪛 LanguageTool
[locale-violation] ~73-~73: In American English, ‘afterward’ is the preferred variant. ‘Afterwards’ is more commonly used in British English and other dialects.
Context: ...ually have to remove this from the list afterwards. Having configured the zones and the o...(AFTERWARDS_US)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
source/_integrations/satel_integra.markdown(2 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_integrations/satel_integra.markdown
[uncategorized] ~62-~62: Did you mean “its” (the possessive pronoun)?
Context: ...option forms. Each partition will have it's own alarm panel, each zone and output w...
(ITS_PREMIUM)
[formatting] ~71-~71: Consider adding a comma after ‘Afterwards’ for more clarity.
Context: .../eba00284-c9fb-4678-9f46-1e67bd806ab3) Afterwards fill in the required fields for each en...
(CONJUNCTIVE_LINKING_ADVERB_COMMA_PREMIUM)
[locale-violation] ~73-~73: In American English, ‘afterward’ is the preferred variant. ‘Afterwards’ is more commonly used in British English and other dialects.
Context: ...ually have to remove this from the list afterwards. Having configured the zones and the o...
(AFTERWARDS_US)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - home-assistant-docs
- GitHub Check: Header rules - home-assistant-docs
- GitHub Check: Pages changed - home-assistant-docs
🔇 Additional comments (2)
source/_integrations/satel_integra.markdown (2)
43-43: Verify the Included Configuration Flow Documentation
The{% include integrations/config_flow.md %}tag on line 43 now pulls in a shared configuration flow section. Please ensure that the content of this included file is fully up to date with the new integration approach and reflects the updated options flow.
45-52: Clarify the “Code” Field in the Simplified Configuration Block
The new{% configuration_basic %}block (lines 45–52) lists fields for Host, Port, and Code. According to the PR objectives and the AI summary, the configuration is intended to be simplified—focusing solely on Host and Port. If the “Code” field is no longer required, consider removing it to align with the updated integration documentation. Otherwise, please update the field’s description to clarify its purpose.
|
|
||
| To get started, select which entry type you want to manage: | ||
|
|
||
|  |
There was a problem hiding this comment.
please upload the images into this repo, under /images/integrations/satel_integra
|
I'm drafting this PR, as changes are needed. Please undraft it, once the changes have been implemented and the PR is ready for review again. |
Proposed change
Update documentation for Satel Integra to reflect the config and options flow.
Type of change
currentbranch).currentbranch).nextbranch).nextbranch).Additional information
Checklist
currentbranch.nextbranch.Summary by CodeRabbit
New Features
Documentation