1088: 'includeDeviceStates' and 'excludeDeviceStates' are no longer supported. Please use 'includeDevices' and 'excludeDevices' going forward.

AlexFilipin / ConditionalAccess

MIT License

266 stars 71 forks source link

1088: 'includeDeviceStates' and 'excludeDeviceStates' are no longer supported. Please use 'includeDevices' and 'excludeDevices' going forward. #17

Closed AlexFilipin closed 3 years ago

AlexFilipin commented 3 years ago

Looking into this, currently seeing an issue that GET via V1 does not get policies with the new elements, planning updates to the JSONs this weekend after its clarified.

AlexFilipin commented 3 years ago

POST also fails against V1, should update JSONs but PATCHING will be broken until V1 returns the policies

AlexFilipin commented 3 years ago

https://github.com/AlexFilipin/ConditionalAccess/commit/b155c59a25cd2cc32431d0c14d9b77f377c83ea8

AlexFilipin commented 3 years ago

https://github.com/AlexFilipin/ConditionalAccess/wiki/Release-history#policy-repository-update-2021-03-03

AlexFilipin commented 3 years ago

Bugfix for policy 500, 501, 505: 'includeDeviceStates' and 'excludeDeviceStates' are no longer supported. Please use 'includeDevices' and 'excludeDevices'. Note: Currently policies with includeDevices/excludeDevices are NOT returned from Graph V1, hence you should only PATCH with Beta until this is fixed.