Skip to content

added another choose example#16972

Merged
frenck merged 8 commits into
home-assistant:currentfrom
Nihvel:patch-1
Mar 15, 2021
Merged

added another choose example#16972
frenck merged 8 commits into
home-assistant:currentfrom
Nihvel:patch-1

Conversation

@Nihvel
Copy link
Copy Markdown
Contributor

@Nihvel Nihvel commented Mar 12, 2021

more choose can be used together for creating extended conditions which do not depend from each others

Proposed change

I added an example that shows how more choose can be used within the same automation or script.

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:

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.

more choose can be used together for creating extended conditions which do not depend from each others
@homeassistant
Copy link
Copy Markdown

Hi @Nihvel,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@probot-home-assistant probot-home-assistant Bot added the current This PR goes into the current branch label Mar 12, 2021
Comment thread source/_docs/scripts.markdown
@frenck frenck added the in-progress This PR/Issue is currently being worked on label Mar 12, 2021
@frenck
Copy link
Copy Markdown
Member

frenck commented Mar 12, 2021

I think the example needs a bit more explanation or comments. Due to its size, it is hard to follow for someone new to the syntax.

Nihvel added 3 commits March 12, 2021 21:07
Extended explanation was added to the example to help understand the automation. The automation was cleaned from unnecessary conditions
pipeline failed due to possible typo
Comment thread source/_docs/scripts.markdown Outdated
Comment thread source/_docs/scripts.markdown Outdated
Comment thread source/_docs/scripts.markdown Outdated
Comment thread source/_docs/scripts.markdown
Nihvel and others added 4 commits March 15, 2021 15:29
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
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, @Nihvel 👍

@frenck frenck merged commit 259fb6f into home-assistant:current Mar 15, 2021
@probot-home-assistant probot-home-assistant Bot removed the in-progress This PR/Issue is currently being worked on label Mar 15, 2021
@Nihvel Nihvel deleted the patch-1 branch March 15, 2021 14:36
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants