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.

Sign up for GitHub

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

Jump to bottom

Add MikroTik BT5 integration documentation #32715

Closed
wants to merge 4 commits into from
Closed

Add MikroTik BT5 integration documentation #32715

wants to merge 4 commits into from

Conversation

Anrijs
Copy link
Contributor

@Anrijs Anrijs commented May 13, 2024

Proposed change

Adds documentation for the MikroTik BT5 integration

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@home-assistant home-assistant bot added has-parent This PR has a parent PR in a other repo next This PR goes into the next branch labels May 13, 2024
@netlify Netlify
Copy link

netlify bot commented May 13, 2024 •
edited
Loading

✅ Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 4a6f792
🔍 Latest deploy log https://app.netlify.com/sites/home-assistant-docs/deploys/664dc5ad690ab600083ec122
😎 Deploy Preview https://deploy-preview-32715--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Anrijs Anrijs mentioned this pull request May 13, 2024
Closed
20 tasks
c0ffeeca7
c0ffeeca7 reviewed May 14, 2024
View reviewed changes
source/_integrations/mikrotik_bt5.markdown Outdated
- [MikroTik TG-BT5-IN](https://mikrotik.com/product/tg_bt5_in/)
- [MikroTik TB-BT5-OUT](https://mikrotik.com/product/tg_bt5_out/)

The MikroTik BT5 integration requires that your MikroTik BT5 devices has MikroTik Beacon enabled. This can be done within the settings portion of the MikroTik Beacon Manager mobile application on both Android and iOS.
Copy link
Contributor

Choose a reason for hiding this comment

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

Would it make sense to place this section before the list of devices?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think device list fits better it it is right under "Suported devices".
Since MikroTik packet format should be enabled by default, app configuration side-note is not as important.

@c0ffeeca7 c0ffeeca7 added the new-integration This PR adds documentation for a new Home Assistant integration label May 14, 2024
c0ffeeca7
c0ffeeca7 reviewed May 14, 2024
View reviewed changes
source/_integrations/mikrotik_bt5.markdown
{% include integrations/config_flow.md %}

The MikroTik BT5 integration will automatically discover devices once the [Bluetooth](/integrations/bluetooth) integration is enabled and functional.

Copy link
Contributor

Choose a reason for hiding this comment

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

could you provide some information on what sensors are supported?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It depens on https://www.home-assistant.io/integrations/bluetooth/ supported devices.
Any bluetooth adapter that can receive other BLE advertisements, like Eddystone or iBeacon, will work.

Copy link
Contributor

Choose a reason for hiding this comment

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

Right, I should have been more specific. :-) What I meant was, what type of sensor entities does this integration provide.

Copy link
Contributor Author

@Anrijs Anrijs May 22, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

I added what sensors each model supports

@frenck
Copy link
Member

frenck commented Jul 30, 2024

Closing PR as the parent PR has been closed.

@frenck frenck closed this Jul 30, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jul 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@c0ffeeca7 c0ffeeca7 c0ffeeca7 left review comments

Assignees
No one assigned
Labels
has-parent This PR has a parent PR in a other repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Anrijs @frenck @c0ffeeca7