Skip to content

Doc topic_template for mqtt.publish#18663

Merged
frenck merged 1 commit into
home-assistant:nextfrom
davidjb:mqtt-topic-template
Nov 30, 2021
Merged

Doc topic_template for mqtt.publish#18663
frenck merged 1 commit into
home-assistant:nextfrom
davidjb:mqtt-topic-template

Conversation

@davidjb
Copy link
Copy Markdown
Contributor

@davidjb davidjb commented Jul 30, 2021

Proposed change

This documents the addition of the topic_template feature for mqtt.publish from home-assistant/core#53743.

Additionally, this slightly rewords the overlapping, existing documentation on this service in order to explain how payload and payload_template work. The current documentation mentions that payload_template would be ignored if payload were specified, but this isn't the case; voluptuous will raise a schema error as these fields are marked as Exclusive.

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

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 has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Jul 30, 2021
This documents the feature from home-assistant/core#53743
and adds an example of its usage.
@davidjb davidjb force-pushed the mqtt-topic-template branch from 9c87604 to 2d5e44c Compare July 30, 2021 15:51
@frenck frenck added the new-feature This PR adds documentation for a new Home Assistant feature to an existing integration label Aug 2, 2021
@frenck frenck added the parent-merged The parent PR has been merged already label Nov 29, 2021
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, @davidjb 👍

@frenck frenck merged commit d3edbad into home-assistant:next Nov 30, 2021
@probot-home-assistant probot-home-assistant Bot removed the parent-merged The parent PR has been merged already label Nov 30, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed 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