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

Use interactive OAuth flow #18

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Use interactive OAuth flow #18

wants to merge 4 commits into from

Conversation

freaktechnik
Copy link
Collaborator

@freaktechnik freaktechnik commented Sep 19, 2022

This uses a custom module to interact with the netatmo API to get access to oauth, as well as making it so the user gets prompted to an oauth flow. This is currently mostly untested and the flow as well as storage of credentials needs some work. Also needs a HTTP route handler for the oauth callback.

src/netatmo.ts Outdated Show resolved Hide resolved
src/adapter.ts Outdated Show resolved Hide resolved
@freaktechnik
Copy link
Collaborator Author

I've updated this with most of the development from the energy adapter work @MichaelKohler has done. I haven't tested any of it yet, since there's still typescript issues.

src/netatmo.ts Show resolved Hide resolved
src/extension.ts Outdated Show resolved Hide resolved
@freaktechnik freaktechnik marked this pull request as ready for review October 16, 2022 11:50
@freaktechnik freaktechnik changed the title WIP: move to using netatmo with actual OAuth Use interactive OAuth flow Oct 16, 2022
@freaktechnik
Copy link
Collaborator Author

I'm now running this on my gateway, since netatmo no longer accepts non-oauthed credentials.

@MichaelKohler
Copy link

I'm now running this as well. Will keep an eye on it and if everything works nicely I'll give this another review.

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

Successfully merging this pull request may close these issues.

2 participants