Skip to content

Updates the Gismeteo custom component to fix compatibility issues with current Home Assistant versions and resolve config flow errors#232

Open
italo-lombardi wants to merge 4 commits intoLimych:devfrom
italo-lombardi:fix-configuration
Open

Updates the Gismeteo custom component to fix compatibility issues with current Home Assistant versions and resolve config flow errors#232
italo-lombardi wants to merge 4 commits intoLimych:devfrom
italo-lombardi:fix-configuration

Conversation

@italo-lombardi
Copy link
Copy Markdown

Proposed change

image

Critical Fixes

  • Fixed config flow 500 error: Removed custom __init__ in OptionsFlowHandler that was attempting to set the read-only config_entry property, which is automatically provided by the base class
  • Replaced deprecated async_timeout: Migrated from async_timeout.timeout to asyncio.timeout (Python 3.11+)
  • Updated cached_property import: Changed from functools.cached_property to propcache.api.cached_property per Home Assistant requirements

Code Quality Improvements

  • Added -> None return type hints to all __init__ methods for pylint compliance
  • Removed unused _update_options method that was no longer called

Type of change

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

Example entry for configuration.yaml:

# Example configuration.yaml

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • The code has been formatted using Black (black --fast custom_components)

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

  • Documentation added/updated to README.md

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.

1 participant