-
Notifications
You must be signed in to change notification settings - Fork 25
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
Repair not raised for MarketingConsentError #587
Comments
Hi, I also encountered the same issue when upgrading the integration. Sascha |
I have the same issue. Workaround doesn't help:
|
I'm having the same issue now as above user:
|
Same here. Using version 1.18.1. |
Report the same issue for version 1.17.1 and 1.19.1. Log files are the same as @Schlichi83 and @miczu71 posted. |
Same here on 1.18.1 |
Hi All, First issue post here, so please be gentle. I think there are two issues being discussed here - the first (and opening issue) may or may not be related, but looks like a consent issue, where-as my (and the subsequent) issues are seemingly a change to the API responses. So apologies for not opening a new issue here. I have the described error showing up since 10:18 today (mashumaro.exceptions.InvalidFieldValue: Field "id" of type UserCapabilityId in UserCapability has invalid value 'MARKETING_CONSENT_GENERIC') The problem seems to be a new id capability being returned by the API, which is not handled well by the data parser. I've made a fix in my local install, and went into my skoda app and slid the slider under Profile -> Permissions & Consent -> Access to Vehicle data (though that might not be related) and things seem to be working again :
Becomes
Restart Hassio, and the API seems to work again. If I could find this file in the repo, I would make a pull request :) |
Where do you change this in HomeAssistant? |
I made this change on the command line in Home Assistant, in the file I mentioned. I would advise caution, as making a bad change here could have non-start consequences for Hassio. As per normal, create a good copy of the file and take care (especially if you're using vi). |
And as if the plea's for help were heard by @WebSpider (Thank you Webspider!) the non-original issue is already fixed in the underlying code, so I am sure will make it's way through the update channels shortly. |
This will be fixed in 1.19.2, awaiting code review now. |
Wow, talking a bout a fast fix. Great job guys! Does anyone have a suggestion on how we can fix this locally without upgrading? I can't seem to find the user.py file on my HA OS... Thanks for the help! |
It is in the myskoda library, not in the integration, not so easy to find. You can force the upgrade by changing myskoda==0.17.x to 0.18.1 in |
Thanks for this detail, I will add it to the next release as well. |
v1.19.2 has been released. which should fix the errors named in #598 and MARKETING_CONSENT_GENERIC as well. |
This did the trick. |
1.19.2 worked for me too. Thank you for that fix! |
Would it be possible to push v1.19..2 as a nor mal update please? |
Hello!
Apologies for bringing this issue up again. I’ve installed the latest version 1.19.1, and when I enter my email and password, I encounter an error. The logs provide the following details:
Am I doing something wrong?
Thank you!
Originally posted by @UZi02 in #453 (comment)
The text was updated successfully, but these errors were encountered: