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

[rotel] Support added for model RX-1052 #16486

Merged
merged 11 commits into from Mar 21, 2024
Merged

Conversation

lolodomo
Copy link
Contributor

@lolodomo lolodomo commented Mar 4, 2024

Signed-off-by: Laurent Garnier lg.hc@free.fr

@lolodomo lolodomo added enhancement An enhancement or new feature for an existing add-on work in progress A PR that is not yet ready to be merged labels Mar 4, 2024
@lolodomo lolodomo marked this pull request as draft March 4, 2024 23:52
@lolodomo
Copy link
Contributor Author

lolodomo commented Mar 9, 2024

Jar for testing:
rotel-RX1052-v1.zip

After downloading, just change the file extension from zip into .jar.

Waiting for Pieter's feedback.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo
Copy link
Contributor Author

New version that should detect tuner input.
rotel-RX1052-v2.zip

Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment from a quick glance over the changes.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo lolodomo marked this pull request as ready for review March 16, 2024 10:24
@lolodomo
Copy link
Contributor Author

Final version for testing:
rotel-RX1052-v3.zip

I will ask Pieter to fully validate this final version.

The PR is now ready for review.

@lolodomo lolodomo removed the work in progress A PR that is not yet ready to be merged label Mar 16, 2024
@lolodomo
Copy link
Contributor Author

Pieter confirmed to me that the last version is working well with its RX-1052 device.

Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Only some minor comments. Otherwise LGTM.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo
Copy link
Contributor Author

@jlaur : changes done.

Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jlaur jlaur merged commit 0d30866 into openhab:main Mar 21, 2024
3 checks passed
@jlaur jlaur added this to the 4.2 milestone Mar 21, 2024
@lolodomo lolodomo deleted the rotel_rx1052 branch March 21, 2024 21:38
magx2 pushed a commit to magx2/openhab2-addons that referenced this pull request Mar 24, 2024
* Two new channels to enable/disable speaker A and speaker B

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants