Skip to content

Add ZWA-2 recover wizard#24

Merged
balloob merged 2 commits into
mainfrom
zwa2-recover
Mar 2, 2026
Merged

Add ZWA-2 recover wizard#24
balloob merged 2 commits into
mainfrom
zwa2-recover

Conversation

@stvncode
Copy link
Copy Markdown
Collaborator

@stvncode stvncode commented Mar 2, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 2, 2026 09:47
@stvncode stvncode self-assigned this Mar 2, 2026
@stvncode stvncode added the enhancement New feature or request label Mar 2, 2026
Comment thread src-11ty/home-assistant-connect-zwa-2/index.md Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new “Recover adapter” step/page to the Home Assistant Connect ZWA-2 section, enabling users to launch the recovery flow via the hosted zwa2-toolbox standalone web component.

Changes:

  • Added a new recover.njk page using the existing install.html layout and embedding the <recover-adapter> web component.
  • Exposed the new recovery page as an action on the ZWA-2 overview page.
  • Updated install.html styling rules to include the new recover-adapter custom element.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src-11ty/home-assistant-connect-zwa-2/recover.njk New recovery page that renders the recovery component and loads its module script.
src-11ty/home-assistant-connect-zwa-2/index.md Adds a “Recover adapter” entry to the actions list linking to the new page.
src-11ty/_includes/install.html Extends existing installer-card CSS selectors to style <recover-adapter> consistently.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@stvncode stvncode requested a review from balloob March 2, 2026 09:53
@AlCalzone
Copy link
Copy Markdown

PR title should be

Add ZWA-2 recover wizard

@balloob balloob changed the title Add zaw2 recover step Add ZWA-2 recover wizard Mar 2, 2026
@balloob balloob merged commit 0b1c98d into main Mar 2, 2026
2 checks passed
@TheJulianJES
Copy link
Copy Markdown
Contributor

The page seems to be slightly broken for me:
image

@balloob
Copy link
Copy Markdown
Member

balloob commented Mar 2, 2026

Probably a stale CSS in the cache.

@TheJulianJES
Copy link
Copy Markdown
Contributor

TheJulianJES commented Mar 2, 2026

The CSS isn't found at all for that subpage. Looks like it's trying to fetch a wrong location? (Screenshot with 404 above)
Can reproduce with all caches cleared and on multiple systems. EDIT: Yeah, the recovery page style is broken for everyone at the moment.

@TheJulianJES
Copy link
Copy Markdown
Contributor

TheJulianJES commented Mar 2, 2026

The page needs to be added to the Vite config rollupOptions here:

toolbox/vite.config.ts

Lines 79 to 128 in 0b1c98d

build: {
outDir: '.',
emptyOutDir: false,
rollupOptions: {
input: {
main: resolve(__dirname, 'dist/index.html'),
homeAssistantVoicePreviewEdition: resolve(
__dirname,
'dist/home-assistant-voice-preview-edition/index.html'
),
homeAssistantVoicePreviewEditionInstall: resolve(
__dirname,
'dist/home-assistant-voice-preview-edition/install/index.html'
),
homeAssistantConnectZwa2: resolve(
__dirname,
'dist/home-assistant-connect-zwa-2/index.html'
),
homeAssistantConnectZwa2Install: resolve(
__dirname,
'dist/home-assistant-connect-zwa-2/install/index.html'
),
homeAssistantConnectZwa2InstallPortable: resolve(
__dirname,
'dist/home-assistant-connect-zwa-2/install-portable/index.html'
),
homeAssistantConnectZwa2UsePoe: resolve(
__dirname,
'dist/home-assistant-connect-zwa-2/use-poe/index.html'
),
homeAssistantConnectZbt1: resolve(
__dirname,
'dist/home-assistant-connect-zbt-1/index.html'
),
homeAssistantConnectZbt1Install: resolve(
__dirname,
'dist/home-assistant-connect-zbt-1/install/index.html'
),
improv: resolve(__dirname, 'dist/improv/index.html'),
homeAssistantConnectZbt2: resolve(
__dirname,
'dist/home-assistant-connect-zbt-2/index.html'
),
homeAssistantConnectZbt2Install: resolve(
__dirname,
'dist/home-assistant-connect-zbt-2/install/index.html'
),
},
},
},

PR:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants