Skip to content

Update configurator.markdown#5774

Merged
DubhAd merged 1 commit into
currentfrom
unknown repository
Jul 12, 2018
Merged

Update configurator.markdown#5774
DubhAd merged 1 commit into
currentfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 12, 2018

The "username" setting was marked as optional, but removing it causes the config to fail the add-on validation test. Changed the note from (optional) to (required).

Description:

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

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.

The "username" setting was marked as optional, but removing it causes the config to fail the add-on validation test. Changed the note from (optional) to (required).
@ghost ghost added the to-do label Jul 12, 2018
Copy link
Copy Markdown
Contributor

@DubhAd DubhAd left a comment

Choose a reason for hiding this comment

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

👍

@DubhAd DubhAd merged commit 08d83f6 into home-assistant:current Jul 12, 2018
@ghost ghost removed the to-do label Jul 12, 2018
@frenck frenck added Fix/Formatting/Language/Spelling current This PR goes into the current branch labels Jul 12, 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.

2 participants