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

Try to fix Unknown Exception - 'list' object has no attribute 'object… #382

Closed
wants to merge 47 commits into from

Conversation

iMicknl
Copy link
Owner

@iMicknl iMicknl commented Dec 8, 2024

…_key' #338

Cruguah and others added 30 commits August 13, 2023 10:52
Added sensor state class of "measurement" to the Nest Protect temperature sensor definition, so that HA recognizes and collects min/mean/max statistical information for these sensors.
Co-authored-by: jnxxx <>
…battery voltage conversions (#283)

Co-authored-by: Ryan McGinty <[email protected]>
* replace depecated Celsius unit

* use event_loop since loop is deprecated

* empty newline at end of file
* Fix battery level sensor Nest Temperature Sensors

This was regressed as part of 54c98d3

SUPPORTED_KEYS is a dictionary using sensor description key as the "key". Prior to this fix, SUPPORTED_KEYS was always picking up the Nest Protect sensor description and completely ignoring the temperature sensor one.

* Fix code style

* Fix code style
iMicknl and others added 17 commits June 15, 2024 23:01
* update pytest-homeassistant-custom-component to match homeassistant

* update pytest-socket

* update pytest-timeout

* update pytest

* unpin pytest-homeassistant-custom-component
* Fixing PynestException: 200 error while subscribing - #345

First step to fix PynestException: 200 error while subscribing

* Fixing PynestException: 200 error while subscribing - #345

Second step to fix PynestException: 200 error while subscribing

* Fixing PynestException: 200 error while subscribing - #345

Third and final step to fix PynestException: 200 error while subscribing

* Fix linting

---------

Co-authored-by: Mick Vleeshouwer <[email protected]>
My fresh installation of Chrome defaulted to "Block all third-party Cookies in Incognito"

Co-authored-by: Mick Vleeshouwer <[email protected]>
* Add Last Audio Self Test and Last Manual Self Test

Adding two more datetime sensors for test times.

* Add Removed from Base sensor

I have chosen tray-remove as icon, as it looks similar to the protect being removed.
@github-actions github-actions bot added the bug Something isn't working label Dec 8, 2024
@tbclark3
Copy link

tbclark3 commented Dec 9, 2024

I'm still getting the error.

@iMicknl
Copy link
Owner Author

iMicknl commented Dec 9, 2024

@tbclark3 with this PR?

@tbclark3
Copy link

tbclark3 commented Dec 9, 2024

No, sorry!! The latest beta came out about the same time as this PR and I mistakenly assumed that it was merged. I now see that it hasn't been. I will test it.

@tbclark3
Copy link

I'm still getting an error with this PR. Log below.

Logger: custom_components.nest_protect
Source: custom_components/nest_protect/init.py:170
integration: Nest Protect (documentation, issues)
First occurred: 4:56:02 PM (1 occurrences)
Last logged: 4:56:02 PM

Unknown exception. Please create an issue on GitHub with your logfile. Updates paused for 5 minutes.
Traceback (most recent call last):
File "/config/custom_components/nest_protect/init.py", line 170, in _async_subscribe_for_data
result = await entry_data.client.subscribe_for_data(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
...<4 lines>...
)
^
File "/config/custom_components/nest_protect/pynest/client.py", line 288, in subscribe_for_data
bucket = Bucket(**bucket)
File "", line 8, in init
File "/config/custom_components/nest_protect/pynest/models.py", line 88, in post_init
self.value = WhereBucketValue(**self.value)
~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^
TypeError: custom_components.nest_protect.pynest.models.WhereBucketValue() argument after ** must be a mapping, not WhereBucketValue

Base automatically changed from beta to main December 30, 2024 11:20
@iMicknl iMicknl closed this Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.