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

[oppo] Add missing input option and enable translations #17783

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

mlobstein
Copy link
Contributor

Add the missing ARC 2 option for the Source Input channel of the BDP-103/BDP-105.
image

Also enable translation of all of the various models' selectable option labels.

@mlobstein mlobstein added the enhancement An enhancement or new feature for an existing add-on label Nov 21, 2024
@@ -105,10 +114,14 @@ public class OppoHandler extends BaseThingHandler implements OppoMessageEventLis
* Constructor
*/
public OppoHandler(Thing thing, OppoStateDescriptionOptionProvider stateDescriptionProvider,
SerialPortManager serialPortManager) {
SerialPortManager serialPortManager, @Reference TranslationProvider translationProvider,
Copy link
Contributor

Choose a reason for hiding this comment

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

Do not use @reference here.

Signed-off-by: Michael Lobstein <[email protected]>
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

LGTM

@lolodomo lolodomo added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Nov 22, 2024
@lolodomo lolodomo merged commit 511fab7 into openhab:main Nov 22, 2024
5 checks passed
@lolodomo lolodomo added this to the 4.3 milestone Nov 22, 2024
@mlobstein mlobstein deleted the oppo_options branch November 22, 2024 16:11
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.

2 participants