Skip to content

Update yolink document#22900

Merged
frenck merged 3 commits intohome-assistant:nextfrom
YoSmart-Inc:next
May 30, 2022
Merged

Update yolink document#22900
frenck merged 3 commits intohome-assistant:nextfrom
YoSmart-Inc:next

Conversation

@matrixd2
Copy link
Copy Markdown
Contributor

@matrixd2 matrixd2 commented May 27, 2022

Proposed change

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

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

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.

@probot-home-assistant probot-home-assistant Bot added 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 May 27, 2022
@probot-home-assistant probot-home-assistant Bot added the next This PR goes into the next branch label May 27, 2022
@probot-home-assistant
Copy link
Copy Markdown

It seems that this PR is targeted against an incorrect branch. Documentation updates which apply to our current stable release should target the current branch. Please change the target branch of this PR to current and rebase if needed. If this is documentation for a new feature, please add a link to that PR in your description.
(message by DocsTargetBranch)

@matrixd2 matrixd2 removed their assignment May 27, 2022
Comment thread source/_integrations/yolink.markdown Outdated

Unless configured otherwise, Home Assistant will use account linking provided by
Nabu Casa for authenticating with YoLink, If you want to use separate credentials, please contact <service@yosmart.com> to obtain a `client_id` and `client_secret`. Then you can add your credentials via application credentials. Settings > Devices & Services > click the menu (three dots at the top right of the screen) and then **Application Credentials**. Enter your credentials in the pop-up window.
<br>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please avoid the use of <br>, it is generally not needed.

Comment thread source/_integrations/yolink.markdown Outdated
@frenck frenck added this to the 2022.6.0 milestone May 30, 2022
@matrixd2 matrixd2 requested a review from frenck May 30, 2022 13:39
Comment thread source/_integrations/yolink.markdown Outdated
@matrixd2 matrixd2 requested a review from frenck May 30, 2022 15:31
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Thanks, @matrixd2 👍

@frenck frenck merged commit 0216a13 into home-assistant:next May 30, 2022
@probot-home-assistant probot-home-assistant Bot removed needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch in-progress This PR/Issue is currently being worked on labels May 30, 2022
@frenck frenck added the cherry-picked This PR has been manually picked and merged into the current branch label May 30, 2022
frenck pushed a commit that referenced this pull request May 30, 2022
@github-actions github-actions Bot locked and limited conversation to collaborators May 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cherry-picked This PR has been manually picked and merged into the current branch cla-signed next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants