Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HA: RF modules discovery as sensors. Subscribe topics, accordingly to (not) defined valueAsATopic. #2117

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

peshovec
Copy link
Contributor

@peshovec peshovec commented Nov 18, 2024

The RF sensors discovery are created with topics like "/433toMQTT"
If valueAsATopic is defined, then published messages are not in the main topic, but in subtree.
Change the topic to be like "/433toMQTT/#" , when valueAsATopic is defined.

Description:

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

…e.g if vaule as topic, then subscribe ... to /#. So in the HA the received message will be populated in the sensors too
@DigiH
Copy link
Collaborator

DigiH commented Nov 18, 2024

Hi @peshovec

Right clicking into the code files and choosing Format Document should get rid of your Code Format error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants