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

Add setup option endpoint #897

Merged
merged 4 commits into from
Jul 9, 2023
Merged

Conversation

iMicknl
Copy link
Owner

@iMicknl iMicknl commented Jun 30, 2023

First implementation of Somfy-Developer/Somfy-TaHoma-Developer-Mode#90 (comment)

  • Add tests
  • get_setup_option_parameter(option, parameter) will return {'errorCode': 'UNSPECIFIED_ERROR', 'error': 'Internal Error'} when option does not exist. If parameter does not exist, it will return None.
  • Add helper for local setup that will return a bool?

@iMicknl iMicknl marked this pull request as draft June 30, 2023 12:11
max_tries=2,
on_backoff=relogin,
)
async def get_setup_option_parameter(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice!

Copy link
Owner Author

Choose a reason for hiding this comment

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

Except for the fact that this throws a very generic and ugly exception on the Overkiz side, if the option does not exist...

@iMicknl iMicknl marked this pull request as ready for review June 30, 2023 20:23
@iMicknl iMicknl requested a review from tetienne July 8, 2023 09:09
@tetienne
Copy link
Collaborator

tetienne commented Jul 9, 2023

About the helper, we haven't such thing yet. Do yourself have any otherwise helpers ideas?

@iMicknl
Copy link
Owner Author

iMicknl commented Jul 9, 2023

@tetienne actually, in previous PRs I added 2 utils (https://github.com/iMicknl/python-overkiz-api/blob/main/pyoverkiz/utils.py) which are some kind of helpers 😁.

@iMicknl iMicknl merged commit 2a9beeb into main Jul 9, 2023
3 checks passed
@iMicknl iMicknl deleted the feature/add_setup_option_endpoint branch July 9, 2023 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants