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

[Feature] Add Support for HKR2 & HKR3 #22

Merged
merged 18 commits into from
Oct 6, 2024
Merged

Conversation

kwithus
Copy link
Contributor

@kwithus kwithus commented Oct 5, 2024

PR adds support for HKR2 & HKR3

@LarsK1 LarsK1 changed the title Kwithus patch 1 [Feature] Add Support for HKR2 & HKR3 Oct 6, 2024
@LarsK1 LarsK1 self-assigned this Oct 6, 2024
@LarsK1 LarsK1 added the enhancement New feature or request label Oct 6, 2024
@LarsK1 LarsK1 linked an issue Oct 6, 2024 that may be closed by this pull request
@LarsK1
Copy link
Owner

LarsK1 commented Oct 6, 2024

Hi @kwithus ,
first of all many thanks for your pr. Unfortunately I can't merge it this way. You've uploaded the new files in the main directory. They should go here: https://github.com/kwithus/hass_solvis_control/tree/kwithus-patch-1/custom_components/solvis_control

Therefore please delete the uploaded files in the main dir and reupload them to the correct directory :)

Thanks a lot!

@LarsK1
Copy link
Owner

LarsK1 commented Oct 6, 2024

I've adjusted the files accordingly for you. Thanks for your work!

@LarsK1 LarsK1 merged commit a1568b9 into LarsK1:main Oct 6, 2024
2 of 3 checks passed
@kwithus
Copy link
Contributor Author

kwithus commented Oct 6, 2024

Thanks for adjusting it! Hopefully I will do a better job next time. ;-)
I will try to suggest more changes.
More important: let’s cross fingers, that Solvis fixes the Modbus issues on their side asap.

@kwithus
Copy link
Contributor Author

kwithus commented Oct 7, 2024 via email

@LarsK1
Copy link
Owner

LarsK1 commented Oct 7, 2024

@kwithus just raise a new PR. Please start the title with "[Feature] Added XYZ" and give a short description about your changes".

Afterwards I will review and give you hints.

@kwithus
Copy link
Contributor Author

kwithus commented Oct 7, 2024

When I try to raise a PR it says nothing to compare and I am not able to initiate a PR
image

@LarsK1
Copy link
Owner

LarsK1 commented Oct 7, 2024

@kwithus you'll need to select your feature-branch not main... (kwithus-patch-1)

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

Successfully merging this pull request may close these issues.

[FEATURE] values for other HK
2 participants