Skip to content

Fix Home Assistant instance credential updates - #54

Draft
SychPL wants to merge 1 commit into
Trooped:mainfrom
SychPL:fix/atomic-ha-credential-update
Draft

Fix Home Assistant instance credential updates#54
SychPL wants to merge 1 commit into
Trooped:mainfrom
SychPL:fix/atomic-ha-credential-update

Conversation

@SychPL

@SychPL SychPL commented Jul 19, 2026

Copy link
Copy Markdown

Summary

  • replace independent URL and token updates with a single connection update
  • validate the candidate URL and token together in both manual and QR flows
  • keep the stored connection unchanged when validation fails
  • add regression tests for paired validation and persistence

Root cause

The settings screen validated a new URL with the stored token and a new token with the stored URL. When switching Home Assistant instances, neither mixed pair could authenticate, so users could not save either value.

User impact

Users can switch to another Home Assistant instance by submitting its URL and long-lived access token together.

Validation

  • ./gradlew :app:testDebugUnitTest
  • ./gradlew :app:assembleDebug

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