PepperDash
[FEATURE]-Samsung MDC Display provide current input feedback
Is your feature request related to a problem? Please describe. From what I can tell, the Samsung MDC device only provides current input feedback reflecting power state. Could someone review and make sure I'm not missing something? I'm utilizing this device in a project and need to show current input selected.
Describe the solution you'd like Current input reflected on an analog input to the bridge.
We have a Samsung plugin that has been more maintained than the Essentials class. I'd use that plugin instead of the built-in functionality.
I'm with @andrew-welker on this one - I feel like we should probably close this issue - but it does bring up the question of why we still maintain the samsung device within essentials when there's a more fully-functional plugin.
I think it is marked as obsolete...and if it's not, it should be with a note to use the EPI. Most of the display classes that are in Essentials proper should be marked that way. We'll remove them in the next major version release.
Is the Samsung plugin public?
Sent from my iPhone
On Mar 15, 2022, at 10:59 PM, Andrew Welker @.***> wrote:
I think it is marked as obsolete...and if it's not, it should be with a note to use the EPI. Most of the display classes that are in Essentials proper should be marked that way. We'll remove them in the next major version release.
— Reply to this email directly, view it on GitHubhttps://github.com/PepperDash/Essentials/issues/778#issuecomment-1068733708, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AENNGSFFAQKQUIHMSGVCTGLVAFTD7ANCNFSM5CBU655Q. You are receiving this because you are subscribed to this thread.Message ID: @.***>
Is the Samsung plugin public?
Yes - the samsung plugin is public and is feature complete and (now) essentials compliant.
I would recommend closing this issue @jtalborough @ndorin