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

Make Android app shortcut use 'Home Assistant' as name. #8261

Merged
merged 1 commit into from
Jun 30, 2017

Conversation

michaelarnauts
Copy link
Contributor

Currently, when you create a shortcut to Home Assistant on Android, it suggests the name "Assistant" instead of "Home Assistant".

I think "Home Assistant" is the correct name for the app to clearly differentiate with Google Assistant or other Assistants.

Check out the animation on this page to see the "Assistant" name.

Copy link
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

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

Thanks 🐦

@fabaff fabaff merged commit 0bcb783 into home-assistant:dev Jun 30, 2017
@balloob balloob mentioned this pull request Jul 1, 2017
balloob added a commit that referenced this pull request Jul 1, 2017
balloob added a commit that referenced this pull request Jul 1, 2017
* Revert "Version bump to 0.49.0.dev0 (#8266)"

This reverts commit 8e4394f.

* Revert "Adding done_message to alert (#8116)"

This reverts commit 5e56bc7.

* Revert "Camera services arm disarm including Netgear Arlo (#7961)"

This reverts commit ed20f7e.

* Revert "Make Android app shortcut use 'Home Assistant' as name instead of just 'Assistant'. (#8261)"

This reverts commit 0bcb783.
@balloob
Copy link
Member

balloob commented Jul 1, 2017

This should not have been merged and needs more discussion. The reason the name is "Assistant" is because otherwise the name will be cutoff on the home screen and it will be "Home Ass…"

@balloob
Copy link
Member

balloob commented Jul 1, 2017

(that's also why the property is called short_name)

balloob added a commit that referenced this pull request Jul 1, 2017
* Revert "Version bump to 0.49.0.dev0 (#8266)"

This reverts commit 8e4394f.

* Revert "Adding done_message to alert (#8116)"

This reverts commit 5e56bc7.

* Revert "Camera services arm disarm including Netgear Arlo (#7961)"

This reverts commit ed20f7e.

* Revert "Make Android app shortcut use 'Home Assistant' as name instead of just 'Assistant'. (#8261)"

This reverts commit 0bcb783.
@michaelarnauts
Copy link
Contributor Author

Hmm, it's "Home Assi.." on default font size here, and "Home Assista.." on small font.

The downside is that Android always uses the short name, like in the task switcher where there is enough space for the full name.

I've gotten used to all the abreviations on the home screen that I haven't noticed it's abreviated. I did notice the "Assistant" name wasn't actually correct.

dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
…e-assistant#8271)

* Revert "Version bump to 0.49.0.dev0 (home-assistant#8266)"

This reverts commit 8e4394f.

* Revert "Adding done_message to alert (home-assistant#8116)"

This reverts commit 5e56bc7.

* Revert "Camera services arm disarm including Netgear Arlo (home-assistant#7961)"

This reverts commit ed20f7e.

* Revert "Make Android app shortcut use 'Home Assistant' as name instead of just 'Assistant'. (home-assistant#8261)"

This reverts commit 0bcb783.
@home-assistant home-assistant locked and limited conversation to collaborators Oct 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants