Skip to content

Reference full service call example#25418

Merged
frenck merged 1 commit into
home-assistant:currentfrom
bcutter:patch-1
Dec 28, 2022
Merged

Reference full service call example#25418
frenck merged 1 commit into
home-assistant:currentfrom
bcutter:patch-1

Conversation

@bcutter
Copy link
Copy Markdown
Contributor

@bcutter bcutter commented Dec 27, 2022

Note the existing example for a usage inside a full service call to avoid confusion with data inside data (following home-assistant/core#84645 (comment))

Proposed change

Helps avoiding confusion with nested use of data:.
Instead of adding an additional full service call example, let's just redirect to the existing one at the end of the page (unfortunately example snippets don't have an anchor which would allow to link to).

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

  • This PR fixes or closes issue: fixes #84645
  • This is my first ever PR. Please advise if handled not correctly so I can improve and contribute better in future.

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.
  • The documentation follows the Home Assistant documentation standards.

Note the existing example for a usage inside a full service call to avoid confusion with data inside data
(following home-assistant/core#84645 (comment))
@home-assistant home-assistant Bot added the current This PR goes into the current branch label Dec 27, 2022
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, @bcutter 👍

@frenck frenck merged commit 738b731 into home-assistant:current Dec 28, 2022
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

current This PR goes into the current branch

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

NFAndroidTV: extra keys not allowed (following documentation example)

2 participants