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

Fix OAuth token retrieval #419

Closed

Conversation

nathangoodman
Copy link

First let me say I have not tested this. Im not a TypeScript developer. Im just submitting this PR to try and learn and because as a user of your plugin I wanted to try and give back a tiny little bit. Im basing this off the comment on the HA plugin thread here: home-assistant/core#101763 (comment)

Thank you for all the work you have done on your plugins. I use them every day and I appreciate all the work you have done on them.

@hjdhjd
Copy link
Owner

hjdhjd commented Oct 12, 2023

Thanks but I'm working on something similar already in my private tree, and will have something to share at some point in the future once I have some confidence in it.

Appreciate the effort and the positive feedback!

@brbeaird
Copy link

brbeaird commented Oct 12, 2023

I actually prefer this PR as a model (simpler changes): https://github.com/Python-MyQ/Python-MyQ/pull/22/files

I did some local testing and verified that will work, but I'm sure hjd's got it handled.

@hjdhjd
Copy link
Owner

hjdhjd commented Oct 12, 2023

I've pushed an update that addresses this - thanks again folks.

@hjdhjd hjdhjd closed this Oct 12, 2023
@github-actions
Copy link

This issue is locked to prevent necroposting on closed issues. Please create a new issue for related discussion, if needed.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants