Skip to content
This repository was archived by the owner on Sep 8, 2024. It is now read-only.

Conversation

@chrisveilleux
Copy link
Member

Description

Fix a bug where the settingsMeta API call was failing on initial boot if the device was paired but the device identity attribute of the API class did not yet know the device's ID

How to test

Boot an unpaired device. There should be no API calls with a URL of v1/device//settingsMeta that fail with a 405 error.

Contributor license agreement signed?

CLA [yes]

…sMeta API call to be issued without a device ID in the URL.
@devs-mycroft devs-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Sep 17, 2019
@forslund
Copy link
Collaborator

Updated the test cases. The 405 is gone. Prepared PR #2313 which is a probable root cause for this issue.

@forslund
Copy link
Collaborator

Oh, and Merging!

@forslund forslund merged commit 35bf4c8 into dev Sep 18, 2019
@forslund forslund deleted the bug/settingsmeta-upload-device-uuid branch September 18, 2019 08:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) Type: Bug - complex

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants