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

Smartthings Binding : adding support for new Smartthings API #17907

Open
lo92fr opened this issue Dec 15, 2024 · 8 comments · May be fixed by #17855
Open

Smartthings Binding : adding support for new Smartthings API #17907

lo92fr opened this issue Dec 15, 2024 · 8 comments · May be fixed by #17855
Labels
enhancement An enhancement or new feature for an existing add-on

Comments

@lo92fr
Copy link
Contributor

lo92fr commented Dec 15, 2024

Smarthings binding is not working since several month because of major change on Smartthings API side.
Previous binding was based on a Smartapp installed on local hub.
The new way to do it should be to used the Smartthings cloud API, installating a Smartapp on the cloud side.

This issue was references at different places 👍

A pull request to rework the binding is currently in work:
Update Smartthings binding for new API #17855

There is still some work to be done, but the new code becoms pretty ready for beta testing.

This new issue is to follow and discuss around the progress of this rewrite.

@lo92fr lo92fr added the enhancement An enhancement or new feature for an existing add-on label Dec 15, 2024
@lo92fr
Copy link
Contributor Author

lo92fr commented Dec 15, 2024

@lolodomo ,

I've create this new issue to follow the work on the Smartthings pull request.
Can you please link it to the pull request.
I'm don't have sufficient access right on my side to do it.

@lo92fr
Copy link
Contributor Author

lo92fr commented Dec 15, 2024

What is working so far / need to be address, as December 15, 2024:

Working :

  • The onboarding process is now pretty end to 99%, far more easier then before as the setup of Smartthings apps is automated in last versions.
  • The state update is now working on both side, from Smartthings to Openhab, and from Openhab to Smarthings.
  • State update from Smarthings to Openhab use new Event system from Smarthings cloud.
  • Typeregistry becomes more complete, and I've get success working with multiple device : Light, Smartplug, Kitchen appliance and others.
  • The converters are now back, and handle most of type conversion between Openhab and Smarthings.

Need to be address:

  • Do some more test, on more device, getting more case handle by converter and type registry.
  • Rewrite the documentation.
  • Code clean-up.

@openhab-bot
Copy link
Collaborator

This issue has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/smartthings-binding-eol/138750/28

@openhab-bot
Copy link
Collaborator

This issue has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/smartthings-bindings/149694/4

@BobRak
Copy link
Contributor

BobRak commented Dec 15, 2024

I have not worked on this binding in several year. I no longer have a configuration I can use to do openhab development. Sorry.

@lo92fr
Copy link
Contributor Author

lo92fr commented Dec 15, 2024

Hello @BobRak,

Thanks for you answer.
happy to see your still there even if not working anymore on this.

Laurent.

@kaikreuzer
Copy link
Member

Great to see that you are working on this, @lo92fr!

I'm don't have sufficient access right on my side to do it.

I have added you to the contributors group, which should hopefully give you more rights.

@lo92fr
Copy link
Contributor Author

lo92fr commented Dec 16, 2024

e added you to the contributors group, which should hopefully give you more ri

Hello Kai (@kaikreuzer),

Thanks for the addition.

Laurent.

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 a pull request may close this issue.

4 participants