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

Update to use http.HTTPStatus instead of const.HTTP_* as per HomeAssistant breaking change #58380 #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions custom_components/ica_shopping_list/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@

import voluptuous as vol

#from homeassistant.const import HTTP_NOT_FOUND, HTTP_BAD_REQUEST

from http import HTTPStatus
from homeassistant.core import callback
from homeassistant.components import http
from homeassistant.components.http.data_validator import RequestDataValidator
Expand Down Expand Up @@ -336,9 +337,10 @@ async def post(self, request, item_id):
request.app["hass"].bus.async_fire(EVENT)
return self.json(item)
except KeyError:
return self.json_message("Item not found", 404)
return self.json_message("Item not found", HTTPStatus.NOT_FOUND)
except vol.Invalid:
return self.json_message("Item not found", 400)
return self.json_message("Item not found", HTTPStatus.BAD_REQUEST)



class CreateShoppingListItemView(http.HomeAssistantView):
Expand Down