-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Adding Fluss docs #38801
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
base: next
Are you sure you want to change the base?
Adding Fluss docs #38801
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
📝 WalkthroughWalkthroughA new documentation file for the Fluss+ integration has been added to Home Assistant. This documentation outlines the prerequisites, setup steps, and features of the integration, which allows users to control Fluss+ devices through button entities. It specifies the need for a Fluss+ account, app, Wi-Fi-connected devices, and an API key. The document also describes the available service for simulating button presses and provides metadata relevant to the integration. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant HomeAssistant
participant FlussAPI
User->>HomeAssistant: Add Fluss+ integration via UI
HomeAssistant->>User: Prompt for API key
User->>HomeAssistant: Enter API key
HomeAssistant->>FlussAPI: Authenticate and fetch devices
FlussAPI-->>HomeAssistant: Return list of devices
HomeAssistant->>User: Display button entities for Fluss+ devices
User->>HomeAssistant: Trigger fluss.press service with entity_id
HomeAssistant->>FlussAPI: Simulate button press on device
FlussAPI-->>HomeAssistant: Confirm action
HomeAssistant-->>User: Service call completes
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (5)
source/_integrations/fluss.markdown (5)
25-25
: Use compound “smartphone” instead of “smart phone.”
Modern English usage writes this as one word.🧰 Tools
🪛 LanguageTool
[style] ~25-~25: Nowadays, it’s more common to write this as one word.
Context: ...ount - The Fluss+ app installed on your smart phone. - Fluss+ device connected to your Wi-F...(RECOMMENDED_COMPOUNDS)
28-28
: Correct nested list indentation under Prerequisites.
The sub-bullet is indented by three spaces instead of two, which may break rendering in some markdown parsers.
Apply:- - You can request an API key in the Fluss+ app under your profile settings. + - You can request an API key in the Fluss+ app under your profile settings.🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
28-28: Unordered list indentation
Expected: 2; Actual: 3(MD007, ul-indent)
32-32
: Ensure headings are surrounded by blank lines.
Add a blank line above## Setup
for better readability and to satisfy markdownlint’s rule on headings.🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
32-32: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below(MD022, blanks-around-headings)
39-39
: Improve clarity and fix grammar in Button description.
- Replace “amount of devices” with “number of devices.”
- Remove duplicate “to.”
- Add a comma after “e.g.”
Suggested rewrite:The buttons represent the number of devices you can trigger via Wi-Fi. For example, if you have 3 devices, you will see 3 buttons.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~39-~39: ‘Amount of’ should usually only be used with uncountable or mass nouns. Consider using “number” if this is not the case.
Context: ... ### Button The buttons represent the amount of devices you have access to to trigge...(AMOUNTOF_TO_NUMBEROF)
[duplication] ~39-~39: Possible typo: you repeated a word.
Context: ...t the amount of devices you have access to to trigger via Wi-Fi. E.g if you have 3 de...(ENGLISH_WORD_REPEAT_RULE)
[uncategorized] ~39-~39: The abbreviation “e.g.” (= for example) requires two periods.
Context: ...ou have access to to trigger via Wi-Fi. E.g if you have 3 devices, you will see 3 b...(E_G)
🪛 markdownlint-cli2 (0.17.2)
39-39: Trailing spaces
Expected: 0 or 2; Actual: 1(MD009, no-trailing-spaces)
41-41
: Add missing word for correct grammar.
Change “as long it's connected” to “as long as it’s connected”:-Each button can be triggered via Wi-Fi as long it's connected to the internet. +Each button can be triggered via Wi-Fi as long as it’s connected to the internet.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
source/_integrations/fluss.markdown
(1 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_integrations/fluss.markdown
[style] ~25-~25: Nowadays, it’s more common to write this as one word.
Context: ...ount - The Fluss+ app installed on your smart phone. - Fluss+ device connected to your Wi-F...
(RECOMMENDED_COMPOUNDS)
[uncategorized] ~39-~39: ‘Amount of’ should usually only be used with uncountable or mass nouns. Consider using “number” if this is not the case.
Context: ... ### Button The buttons represent the amount of devices you have access to to trigge...
(AMOUNTOF_TO_NUMBEROF)
[duplication] ~39-~39: Possible typo: you repeated a word.
Context: ...t the amount of devices you have access to to trigger via Wi-Fi. E.g if you have 3 de...
(ENGLISH_WORD_REPEAT_RULE)
[uncategorized] ~39-~39: The abbreviation “e.g.” (= for example) requires two periods.
Context: ...ou have access to to trigger via Wi-Fi. E.g if you have 3 devices, you will see 3 b...
(E_G)
🪛 markdownlint-cli2 (0.17.2)
source/_integrations/fluss.markdown
28-28: Unordered list indentation
Expected: 2; Actual: 3
(MD007, ul-indent)
32-32: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below
(MD022, blanks-around-headings)
39-39: Trailing spaces
Expected: 0 or 2; Actual: 1
(MD009, no-trailing-spaces)
⏰ 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/fluss.markdown (2)
4-7
: Verify metadata fields against Home Assistant standards.
Ensure thatha_category
,ha_iot_class
, andha_release
use the exact values and formatting required by Home Assistant’s documentation conventions (e.g., valid category taxonomy, version string format, and snake_case for IoT class).
49-52
: Verify that all service parameters are documented.
Onlyentity_id
is listed forfluss.press
. Confirm whether additional options (such as atarget
for multiple devices or other payload fields) exist and include them in the table if so.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
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.
Thank you, @Marcello17 👍
Proposed change
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit
fluss.press
service in Home Assistant.