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

Fix hass setting on stack #1868

Merged
merged 9 commits into from
Oct 27, 2018
Merged

Fix hass setting on stack #1868

merged 9 commits into from
Oct 27, 2018

Conversation

balloob
Copy link
Member

@balloob balloob commented Oct 26, 2018

No longer set hass on a card in a stack if it's undefined.

@ghost ghost assigned balloob Oct 26, 2018
@ghost ghost added the in progress label Oct 26, 2018
@thomasloven thomasloven added this to the 0.81.1 milestone Oct 26, 2018
element.hass = this._hass;
}
}
import { HuiStackCard } from "./hui-stack-card.js";
Copy link
Contributor

@thomasloven thomasloven Oct 26, 2018

Choose a reason for hiding this comment

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

Remove the .js. It's a typescript file.

@balloob balloob changed the base branch from master to dev October 27, 2018 08:54
@balloob balloob merged commit 8b02371 into dev Oct 27, 2018
@ghost ghost removed the in progress label Oct 27, 2018
@balloob balloob deleted the fix-stack-hass-setting branch October 27, 2018 10:00
balloob added a commit that referenced this pull request Oct 27, 2018
* Fix hass setting on stack

* Don't set hass on pic elements if undefined

* Don't set hass on entity rows if undefined

* prefix config prop

* Pic elements set hass yoooo

* Remove interface

* Make stack config private

* Fix import

* Lint
balloob pushed a commit that referenced this pull request Oct 29, 2018
* Thermostat fix (#1867)

* Thermostat fix

* Fix for unknown operation mode

* No title for you

* I suck at removing unnecessary things

* Fixing pointless Ternary operator

* Add version bump script

* Fix hass setting on stack (#1868)

* Fix hass setting on stack

* Don't set hass on pic elements if undefined

* Don't set hass on entity rows if undefined

* prefix config prop

* Pic elements set hass yoooo

* Remove interface

* Make stack config private

* Fix import

* Lint

* Bumped version to 20181026.1

* 🔨 move not text to header and name to secondary text

* 🔨 reduce some other margins
@exetico
Copy link

exetico commented Oct 31, 2018

Could this be a problem with vertical-stack too?
https://community.home-assistant.io/t/0-81-new-lovelace-cards-lg-soundbars-dyson-fans-elk-m1/75277/92?u=exetico

hui-conditional-card TypeError: "hass is undefined"
	hass https://mydomain.com/local/mini-media-player/mini-media-player.js:20:11
	setConfig https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:418:1949
	create_card_element_createElement https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1933:2748
	createCardElement https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1933:3897
	_cards https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1131:1071
	setConfig https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1131:1033
	create_card_element_createElement https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1933:2748
	createCardElement https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1933:3897
	elements https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1995:1068
	_createCards https://mydomain.com/frontend_latest/8avalueremovedd749b3.chunk.js:1995:1024
8avalueremovedd749b3.chunk.js:1933

@iantrich
Copy link
Member

@exetico fixed in #1868 for stacks

@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2022
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.

7 participants