Skip to content

Geo Location Trigger to HA - Form#11644

Merged
zsarnett merged 9 commits intodevfrom
geo-location-ha-form
Feb 10, 2022
Merged

Geo Location Trigger to HA - Form#11644
zsarnett merged 9 commits intodevfrom
geo-location-ha-form

Conversation

@zsarnett
Copy link
Copy Markdown
Contributor

Breaking change

Proposed change

image

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@zsarnett zsarnett added the needs design preview PRs with this label will trigger a GitHub action to generate a gallery preview label Feb 10, 2022
return;
}

this._schema = [
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Generate schema using memoizeOne that takes localize: LocalizeFunc. That way when translations update, the schema updates too.

balloob
balloob previously approved these changes Feb 10, 2022
…er-geo_location.ts

Co-authored-by: Paulus Schoutsen <balloob@gmail.com>

protected render() {
const { source, zone, event } = this.trigger;
if (!this._schema) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this is not possible

…er-geo_location.ts

Co-authored-by: Paulus Schoutsen <balloob@gmail.com>
balloob
balloob previously approved these changes Feb 10, 2022
@zsarnett zsarnett merged commit 9912d42 into dev Feb 10, 2022
@delete-merged-branch delete-merged-branch bot deleted the geo-location-ha-form branch February 10, 2022 22:12
@github-actions github-actions bot locked and limited conversation to collaborators Feb 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed needs design preview PRs with this label will trigger a GitHub action to generate a gallery preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants