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 note and example about RouterOS user rights + terminology fixes #7615

Merged
merged 20 commits into from
Dec 26, 2018
Merged

Conversation

soosp
Copy link
Contributor

@soosp soosp commented Nov 24, 2018

Description:

Pull request in home-assistant (if applicable): home-assistant/home-assistant#

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

@ghost ghost added the to-do label Nov 24, 2018
@ghost ghost assigned fabaff Nov 24, 2018
@ghost ghost added ready-for-review This PR needs to be reviewed and removed to-do labels Nov 24, 2018
Copy link
Contributor Author

@soosp soosp left a comment

Choose a reason for hiding this comment

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

Thanks for the enhancements!

source/_components/device_tracker.mikrotik.markdown Outdated Show resolved Hide resolved
source/_components/device_tracker.mikrotik.markdown Outdated Show resolved Hide resolved
@ghost ghost assigned frenck Dec 26, 2018
@frenck frenck added enhancement current This PR goes into the current branch labels Dec 26, 2018
@frenck
Copy link
Member

frenck commented Dec 26, 2018

Thank you @soosp! That looks great! 👍
That deserves a medal 🥇

@frenck frenck merged commit 572b1c0 into home-assistant:next Dec 26, 2018
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Dec 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current This PR goes into the current branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants