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

0.59.1 #10936

Merged
merged 5 commits into from
Dec 4, 2017
Merged

0.59.1 #10936

merged 5 commits into from
Dec 4, 2017

Conversation

balloob
Copy link
Member

@balloob balloob commented Dec 4, 2017

balloob and others added 5 commits December 3, 2017 19:36
* Fixed icon path, added dynamic icon

* Addressing requested changes

* Using DEFAULT_ICON

* Using CONF_ICON from const

* Getting hass_frontend path via import

* Lint

* Using embedded 1px transparent icon

* woof -.-

* Lint
* check for none

* fix error from no store being set

* typo

* Lint

* fix default as per notes. Lint fix and make closest store None not False

* update default
@balloob balloob requested review from andrey-git and a team as code owners December 4, 2017 03:44
@homeassistant homeassistant added integration: dominos integration: ios merging-to-master This PR is merging into the master branch and should probably change the branch to `dev`. platform: notify.nfandroidtv cla-signed labels Dec 4, 2017
@balloob balloob merged commit 015cdd1 into master Dec 4, 2017
@balloob balloob deleted the release-0-59-1 branch December 4, 2017 04:08
@home-assistant home-assistant locked and limited conversation to collaborators Mar 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed integration: dominos integration: ios integration: nfandroidtv merging-to-master This PR is merging into the master branch and should probably change the branch to `dev`.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants