Skip to content

Update bmw_connected_drive.markdown#6024

Merged
frenck merged 3 commits intohome-assistant:nextfrom
Alexxander0:next
Aug 17, 2018
Merged

Update bmw_connected_drive.markdown#6024
frenck merged 3 commits intohome-assistant:nextfrom
Alexxander0:next

Conversation

@Alexxander0
Copy link
Copy Markdown
Contributor

@Alexxander0 Alexxander0 commented Aug 15, 2018

Description:
Update of documentation of the BMW connected drive platform

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

Checklist:

  • Branch: Fixes, changes and adjustments should be created against current. New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

@ghost ghost added the to-do label Aug 15, 2018
@frenck frenck added has-parent This PR has a parent PR in another repo needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch new-feature This PR adds documentation for a new Home Assistant feature to an existing integration in-progress This PR/Issue is currently being worked on and removed to-do labels Aug 15, 2018
@frenck
Copy link
Copy Markdown
Member

frenck commented Aug 15, 2018

Hi there @Alexxander0!

Thank you for this PR! Unfortunately, it is targeted against the wrong base. Since this is a new feature (implemented in your parent PR), it will be part of a next release, so it should go into the next branch.

Please update the target branch and rebase if needed.

@Alexxander0 Alexxander0 changed the base branch from current to next August 15, 2018 21:25
services:
description: Enables or disables the services of the vehicles including the lock. Default true (Services enabled)
read_only:
description: Enables or disables the services of the vehicles including the lock. Default false (Services enabled)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please use the default: key to clarify what the default is -> documentation.

@frenck frenck added ready-for-review This PR needs to be reviewed next This PR goes into the next branch and removed in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Aug 16, 2018
@frenck frenck assigned frenck and unassigned Alexxander0 Aug 16, 2018
@frenck
Copy link
Copy Markdown
Member

frenck commented Aug 17, 2018

Thanks, @Alexxander0! 🏅

@frenck frenck merged commit 1bdb25d into home-assistant:next Aug 17, 2018
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Aug 17, 2018
flo-wer pushed a commit to flo-wer/home-assistant.io that referenced this pull request Aug 17, 2018
* Update bmw_connected_drive.markdown

* Update bmw_connected_drive.markdown

* Update bmw_connected_drive.markdown
flo-wer pushed a commit to flo-wer/home-assistant.io that referenced this pull request Aug 17, 2018
* Update bmw_connected_drive.markdown

* Update bmw_connected_drive.markdown

* Update bmw_connected_drive.markdown
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo new-feature This PR adds documentation for a new Home Assistant feature to an existing integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants