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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate hide_if_away from device_tracker #30833

Merged
merged 1 commit into from Jan 16, 2020
Merged

Deprecate hide_if_away from device_tracker #30833

merged 1 commit into from Jan 16, 2020

Conversation

@frenck
Copy link
Member

frenck commented Jan 16, 2020

Breaking Change:

The hide_if_away configuration option for device trackers has been deprecated and pending for removal in Home Assistant 0.107.0.

Please ensure this configuration option is not used in your configuration and isn't present in the, also deprecated, known_devices.yaml file.

Description:

See breaking changes 馃槈

Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#11766

Example entry for configuration.yaml (if applicable):

device_tracker:
  - platform: netgear
    host: IP_ADDRESS
    username: YOUR_USERNAME
    interval_seconds: 10
    consider_home: 180
    new_device_defaults:
      track_new_devices: true
      hide_if_away: true

Example entry for known_devices.yaml:

devicename:
  name: Friendly Name
  mac: EA:AA:55:E7:C6:94
  picture: https://www.home-assistant.io/images/favicon-192x192.png
  track: true
  hide_if_away: false

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist

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

@project-bot project-bot bot added this to Needs review in Dev Jan 16, 2020
@frenck frenck mentioned this pull request Jan 16, 2020
2 of 2 tasks complete
@frenck frenck marked this pull request as ready for review Jan 16, 2020
Dev automation moved this from Needs review to Reviewer approved Jan 16, 2020
@balloob balloob merged commit 8861c80 into dev Jan 16, 2020
9 checks passed
9 checks passed
CI Build #20200116.24 succeeded
Details
CI (FullCheck Mypy) FullCheck Mypy succeeded
Details
CI (FullCheck Pylint) FullCheck Pylint succeeded
Details
CI (Overview CheckFormat) Overview CheckFormat succeeded
Details
CI (Overview Lint) Overview Lint succeeded
Details
CI (Overview Validate) Overview Validate succeeded
Details
CI (Tests PyTest Python37) Tests PyTest Python37 succeeded
Details
cla-bot Everyone involved has signed the CLA
docs-missing Documentation ok.
Dev automation moved this from Reviewer approved to Done Jan 16, 2020
@delete-merged-branch delete-merged-branch bot deleted the frenck-2020-0092 branch Jan 16, 2020
@lock lock bot locked and limited conversation to collaborators Jan 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Dev
  
Done
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can鈥檛 perform that action at this time.