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

Add missing phone permission to Wear manifest #3200

Merged

Conversation

dshokouhi
Copy link
Member

Summary

Fixes: #3197 as I had forgotten to add this permission to the Wear manifest

Screenshots

Link to pull request in Documentation repository

Documentation: home-assistant/companion.home-assistant#

Any other notes

@dshokouhi
Copy link
Member Author

end user confirmation from a real device the fix worked: #3197 (comment)

we can ignore the unknown value as its a known core issue home-assistant/core#79837

Copy link
Member

@jpelgrom jpelgrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find!

I've checked the requiredPermissions(sensorId: String) functions for sensors in common and wear and everything is in the manifest now.

@JBassett JBassett merged commit 527325f into home-assistant:master Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Three sensors cannot be activated in mobile app
3 participants