Guy293 / Tami4EdgeAPI

Tami 4 Edge / Edge+ API In Python
https://pypi.org/project/Tami4EdgeAPI/
MIT License
16 stars 2 forks source link

Tami4 integration stopped working #7

Closed NiiiYaa closed 3 months ago

NiiiYaa commented 3 months ago

First the 'boiled water' button stopped working (once clicked it generated an error). After few failed attempts I wanted to 're-install', the I deleted the integration and tried to install. On the validation/verification process (SMS) - after inputting the mobile number - "Unexpected error" is received (screenshot below). 2024-06-04 08_12_14-Settings – Home Assistant

Here is the log: Logger: homeassistant.components.tami4.config_flow Source: components/tami4/config_flow.py:46 integration: Tami4 Edge / Edge+ (documentation, issues) First occurred: 8:08:17 AM (9 occurrences) Last logged: 10:53:03 AM

Unexpected exception Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/components/tami4/config_flow.py", line 46, in async_step_user await self.hass.async_add_executor_job( File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/Tami4EdgeAPI/Tami4EdgeAPI.py", line 215, in request_otp if not response["success"]:


KeyError: 'success'
Guy293 commented 3 months ago

Hi, by looking at the logs attached in: https://github.com/home-assistant/core/issues/113167#issuecomment-2146682339 Looks like they changed the API. I will fix it soon.

michaelrodov commented 3 months ago

@Guy293 thanks guy was about to post here, but saw that you responded already.

shaikedem commented 3 months ago

following

danielrubin1989 commented 3 months ago

Thanks @Guy293 , Great integration 👍 following..

Guy293 commented 3 months ago

I've released an update for Tami4EdgeAPI, version v3.0. I am now working on pushing it to Home Assistant repo.

michaelrodov commented 3 months ago

I've released an update for Tami4EdgeAPI, version v3.0. I am now working on pushing it to Home Assistant repo.

thank you Guy! אין עליך!

Guy293 commented 3 months ago

I've opened a PR In home assistant, Y'all can help merging it faster by reviewing it and adding a comment: https://github.com/home-assistant/core/pull/119158

michaelrodov commented 2 months ago

@Guy293 which version of HA it will be released on? thought its part of 2024.6.2?

shaikedem commented 2 months ago

any update on this? its stiull not working ...

Guy293 commented 2 months ago

any update on this? its stiull not working ...

Seems like the latest home assistant version includes the fix

orenc75 commented 2 months ago

Hi Guy, I updated to the latest hass version and still encounter a problem. I get "Unknown error occurred" once I enter the code I get via SMS. image

I am sure my phone number is registered since I am using the Tami4 app and I am sure I write the correct code into hass...

Here is the log: Logger: root Source: /usr/local/lib/python3.12/site-packages/Tami4EdgeAPI/Tami4EdgeAPI.py:251 First occurred: July 3, 2024 at 9:45:08 PM (3 occurrences) Last logged: 7:39:21 AM

OTP Submission Failed, response: {'status': 403, 'message': 'invalid_grant', 'description': 'Wrong phone number or verification code.', 'timestamp': '2024-07-03T18:45:08Z'} OTP Submission Failed, response: {'status': 403, 'message': 'invalid_grant', 'description': 'Wrong phone number or verification code.', 'timestamp': '2024-07-03T18:45:44Z'} OTP Submission Failed, response: {'status': 403, 'message': 'invalid_grant', 'description': 'Wrong phone number or verification code.', 'timestamp': '2024-07-04T04:39:21Z'}