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

Remove hard coded 5° in debug message #1603

Closed
wants to merge 1 commit into from

Conversation

bell07
Copy link

@bell07 bell07 commented Feb 25, 2025

Motivation:

Found this hard coded debug message and took it as an opportunity for first contribution

Changes:

Replace hard-coded 5° by used min_temp value

Related issue (check one):

  • fixes #
  • There is no related issue ticket

Checklist (check one):

  • I did not change any code (e.g. documentation changes)
  • The code change is tested and works locally.

Test-Hardware list (for code changes)

Docker installation on amd64 Gentoo host.

HA Version: 2025.2.5
Zigbee2MQTT Version: -
TRV Hardware:

  • Danfoss "Keemple smart radiator" (Z-Wave.js)
  • Eurotronic "SPZB0001" (Zigbee)

New device mappings

No

@folfy
Copy link
Collaborator

folfy commented Mar 8, 2025

Thanks!

@folfy
Copy link
Collaborator

folfy commented Mar 8, 2025

@KartoffelToby Can you merge this PR, despite requiring commits to have verified signatures? I don't have permission to overrule this :/

@KartoffelToby
Copy link
Owner

@bell07 thanks, i patched it because of your missing signed commit

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.

3 participants