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

Add dev blog about deprecating @bind_hass and hass.components #2093

Merged

Conversation

jpbede
Copy link
Member

@jpbede jpbede commented Feb 27, 2024

Proposed change

Add a dev blog about deprecating the @bind_hass decorator and the use of hass.components.
Also fix some code examples that use hass.components.

Logging PR in core: home-assistant/core#111508

Related home-assistant/core#111522 home-assistant/core#111494

Type of change

  • Document existing features within Home Assistant
  • Document new or changing features which there is an existing pull request elsewhere
  • Spelling or grammatical corrections, or rewording for improved clarity
  • Changes to the backend of this documentation
  • Removed stale or deprecated documentation

Additional information

  • This PR fixes or closes issue: fixes #
  • Link to relevant existing code or pull request:

Copy link
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

See above.

@home-assistant
Copy link

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@home-assistant home-assistant bot marked this pull request as draft February 27, 2024 13:07
@jpbede jpbede marked this pull request as ready for review February 27, 2024 13:49
MartinHjelmare
MartinHjelmare previously approved these changes Feb 27, 2024
Copy link
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Thanks!

MartinHjelmare
MartinHjelmare previously approved these changes Feb 27, 2024
MartinHjelmare
MartinHjelmare previously approved these changes Feb 28, 2024
Co-authored-by: Martin Hjelmare <[email protected]>
Copy link
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Thanks!

@MartinHjelmare MartinHjelmare merged commit a6a6b40 into home-assistant:master Mar 4, 2024
4 checks passed
@jpbede jpbede deleted the deprecate-bind-hass-components branch March 4, 2024 09:15
@UK-JustinSmall
Copy link

Still getting these warnings in the HA logs and on latest version how do I fix this??????

Logger: homeassistant.helpers.frame
Source: helpers/frame.py:156
First occurred: 27 March 2024 at 02:33:04 (1 occurrences)
Last logged: 27 March 2024 at 02:33:04

Detected that custom integration 'hacs' accesses hass.components.frontend. This is deprecated and will stop working in Home Assistant 2024.9, it should be updated to import functions used from frontend directly at custom_components/hacs/frontend.py, line 68: hass.components.frontend.async_register_built_in_panel(, please create a bug report at https://github.com/hacs/integration/issues

@jpbede
Copy link
Member Author

jpbede commented Mar 29, 2024

A closed PR shouldn’t be used for support. Please check the issue at the HACS repository hacs/integration#3534

@UK-JustinSmall
Copy link

It's locked for comments - this is the only one I see where I can add to what is known as an existing issue. Seems it's known but not fixed or no one is telling us what needs to be done to fix it. That's why I am commenting here jpbede. Sorry.

@jpbede
Copy link
Member Author

jpbede commented Mar 29, 2024

There ist nothin you need to do but to be patient and wait for a release of HACS, they know about this and there is also already a PR for this, which you commented on. Also there are 4 releases until this would be a breaking change, so enough of time 🤷‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants