jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-04-28 03:07:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove suppress_warning from config options for vizio component (#30536)

Browse Source 
* move constants, move schema validation to init, prepare for zeroconf/config flow

* mistake in refactor

* update .coveragerc, move validate_auth to __init__, only attempt to get ESN if device setup is validated

* fix schema validation

* remove suppress warning config option

* remove config_flow from manifest since its not ready yet

* update requirements_all.txt

* fix issue with merge conflict
This commit is contained in:
Raman Gupta 2020-01-07 07:31:13 -05:00 committed by Paulus Schoutsen
parent 73af75cb41
commit a9f6dd698e
5 changed files with 3 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
homeassistant/components/vizio/__init__.py
View File

@ -10,7 +10,6 @@ from homeassistant.const import (
from homeassistant.helpers import config_validation as cv from homeassistant.helpers import config_validation as cv
from .const import ( from .const import (
CONF_SUPPRESS_WARNING,
CONF_VOLUME_STEP, CONF_VOLUME_STEP,
DEFAULT_DEVICE_CLASS, DEFAULT_DEVICE_CLASS,
DEFAULT_NAME, DEFAULT_NAME,
@ -33,7 +32,6 @@ VIZIO_SCHEMA = {
vol.Required(CONF_HOST): cv.string, vol.Required(CONF_HOST): cv.string,
vol.Optional(CONF_ACCESS_TOKEN): cv.string, vol.Optional(CONF_ACCESS_TOKEN): cv.string,
vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string,
vol.Optional(CONF_SUPPRESS_WARNING, default=False): cv.boolean,
vol.Optional(CONF_DEVICE_CLASS, default=DEFAULT_DEVICE_CLASS): vol.All( vol.Optional(CONF_DEVICE_CLASS, default=DEFAULT_DEVICE_CLASS): vol.All(
cv.string, vol.Lower, vol.In(["tv", "soundbar"]) cv.string, vol.Lower, vol.In(["tv", "soundbar"])
), ),

1
homeassistant/components/vizio/const.py
View File

@ -1,6 +1,5 @@
"""Constants used by vizio component.""" """Constants used by vizio component."""
CONF_SUPPRESS_WARNING = "suppress_warning"
CONF_VOLUME_STEP = "volume_step" CONF_VOLUME_STEP = "volume_step"
DEFAULT_NAME = "Vizio SmartCast" DEFAULT_NAME = "Vizio SmartCast"

2
homeassistant/components/vizio/manifest.json
View File

@ -3,7 +3,7 @@
"name": "Vizio", "name": "Vizio",
"documentation": "https://www.home-assistant.io/integrations/vizio", "documentation": "https://www.home-assistant.io/integrations/vizio",
"requirements": [ "requirements": [
"pyvizio==0.0.11" "pyvizio==0.0.12"
], ],
"dependencies": [], "dependencies": [],
"codeowners": ["@raman325"] "codeowners": ["@raman325"]

15
homeassistant/components/vizio/media_player.py
View File

@ -3,7 +3,6 @@ from datetime import timedelta
import logging import logging
from pyvizio import Vizio from pyvizio import Vizio
from requests.packages import urllib3
import voluptuous as vol import voluptuous as vol
from homeassistant import util from homeassistant import util
@ -28,13 +27,7 @@ from homeassistant.const import (
) )
from . import VIZIO_SCHEMA, validate_auth from . import VIZIO_SCHEMA, validate_auth
from .const import ( from .const import CONF_VOLUME_STEP, DEFAULT_NAME, DEVICE_ID, ICON
CONF_SUPPRESS_WARNING,
CONF_VOLUME_STEP,
DEFAULT_NAME,
DEVICE_ID,
ICON,
)
_LOGGER = logging.getLogger(__name__) _LOGGER = logging.getLogger(__name__)
@ -78,12 +71,6 @@ def setup_platform(hass, config, add_entities, discovery_info=None):
) )
return return
if config[CONF_SUPPRESS_WARNING]:
_LOGGER.warning(
"InsecureRequestWarning is disabled "
"because of Vizio platform configuration"
)
urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)
add_entities([device], True) add_entities([device], True)

2
requirements_all.txt
View File

@ -1696,7 +1696,7 @@ pyversasense==0.0.6
pyvesync==1.1.0 pyvesync==1.1.0
# homeassistant.components.vizio # homeassistant.components.vizio
pyvizio==0.0.11 pyvizio==0.0.12
# homeassistant.components.velux # homeassistant.components.velux
pyvlx==0.2.12 pyvlx==0.2.12