fix: CWOP skipped in options flow + Blitzortung discovery + FR translations (closes #83, #81, improves #75)#85
Merged
Conversation
…tzortung discovery, FR translations Fixes #83: async_step_pwsweather, async_step_wow, async_step_awekas, and async_step_owm_stations were each missing steps further down the upload chain (OWM stations, Windy, CWOP). When PWSWeather + CWOP were both enabled the options flow jumped directly to alerts after PWSWeather, silently skipping CWOP credential entry and leaving the uploader unconfigured. Improves #75: _discover_blitzortung now also matches by entity_id prefix (sensor.blitzortung_*) in addition to platform name, and the count pattern now checks for "count" in entity_id (not just "counter"). Covers forks or renamed integrations that don't set platform="blitzortung". Closes #81: incorporate Benjamin's French translation additions (soil sensors, conditions_summary, forecast_brier, nowcast_confidence, 90d anomaly sensors, terminology improvements) with corrected translations for soil_moisture name and deficit attribute (were left in English) and fixed missing closing parenthesis in enable_soil descriptions. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes Unable to feed PWSWeather and CWOP simultaneously #83 — Upload service options flow skipped CWOP (and other later steps) when PWSWeather was enabled alongside it. Four step handlers (
pwsweather,wow,awekas,owm_stations) each had an incomplete next-step chain that jumped to MQTT/alerts before reaching OWM Stations → Windy → CWOP. Each handler now walks the full remaining chain.Improves Lightning nearest distance (km, e.g. from WH57) shows “Entity not found” in Home Assistant #75 — Blitzortung auto-discovery now also matches entities by
sensor.blitzortung_*entity ID prefix (not justplatform == "blitzortung"), covering renamed/forked installs. Also adds"count" in eidto the counter pattern (previously only"counter" in eidwas checked). Issue reopened to request clarification from reporter.Closes Update French translations for weather sensors #81 — Incorporates Benjamin's French translation PR with corrections:
soil_moisturename anddeficit_pctattribute were left in English ("Soil Moisture", "Moisture deficit (%)") — translated to French. Twoenable_soildescriptions were missing a closing).Test plan
python -m json.tool fr.json🤖 Generated with Claude Code