Skip to content

ui: add Connect button to Overview for credential changes#385

Closed
wizaj wants to merge 1 commit intoopenclaw:mainfrom
wizaj:ui-add-connect-button
Closed

ui: add Connect button to Overview for credential changes#385
wizaj wants to merge 1 commit intoopenclaw:mainfrom
wizaj:ui-add-connect-button

Conversation

@wizaj
Copy link

@wizaj wizaj commented Jan 7, 2026

Summary

Adds a dedicated Connect button to the Overview page to apply gateway credential changes (URL, token, or password).

Problem

The existing Refresh button was confusing - it displayed the text "Reconnect to apply changes" but only refreshed data without actually reconnecting to the gateway. This made password authentication unusable in the UI, as there was no way to apply the password without manually triggering a reconnect via the browser console.

Solution

  • Added a new Connect button that calls state.connect() to reconnect the gateway client
  • Kept the Refresh button for refreshing overview data
  • Updated the helper text to "Click Connect to apply credential changes" for clarity
  • Added proper spacing between buttons with gap: 8px

Testing

Tested with password authentication on tailscale funnel mode:

  1. Entered password in the "Password (not stored)" field
  2. Clicked the Connect button
  3. Connection status changed to "Connected" and authentication succeeded
  4. Overview page data loaded successfully

Files Changed

  • ui/src/ui/app-render.ts: Added onConnect callback that triggers state.connect()
  • ui/src/ui/views/overview.ts: Added Connect button to UI and updated props type

Fixes the password authentication workflow in the control panel.

The Refresh button in the Overview page was confusing - it said
'Reconnect to apply changes' but only refreshed data without
reconnecting. This adds a dedicated Connect button that calls
state.connect() to reconnect the gateway client with updated
credentials (URL, token, or password).

Fixes the issue where password authentication couldn't be applied
without manually triggering a reconnect via console.
@steipete
Copy link
Contributor

steipete commented Jan 8, 2026

Rebased onto main, resolved UI conflict (Connect button + updated copy), added changelog entry, ran pnpm lint/build/test, and fast-forwarded main. Landed as e9346e6. Thanks @wizaj!

@wizaj wizaj deleted the ui-add-connect-button branch January 8, 2026 22:01
@wizaj
Copy link
Author

wizaj commented Jan 8, 2026

My first ever PR 🥺 honoured!

dgarson added a commit to dgarson/clawdbot that referenced this pull request Feb 9, 2026
* fix: update Claude task URL format to /code/ path

Claude web task URLs use the format https://claude.ai/code/session_{id}
rather than https://claude.ai/chat/{id}. Updated the base URL in the
canonicalizer, builder, and tests accordingly.

* more work queue/work-item changes

* CRN: preserve case for external task IDs
zooqueen pushed a commit to hanzoai/bot that referenced this pull request Mar 6, 2026
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.

2 participants