Skip to content

EzyChat: per-chat figures in the Add dialog before the sign-in; release 0.1.8 - #35

Merged
jazztong merged 2 commits into
mainfrom
fix/ezychat-dialog
Nov 2, 2026
Merged

jazztong merged 2 commits into
mainfrom
fix/ezychat-dialog

Conversation

@jazztong

@jazztong jazztong commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Problem

Before EzyChat was connected, the Add EzyChat dialog showed "Counting chats..." under every history choice for ever. The page asked the server for the estimate only when EzyChat was already connected, so the value never arrived. It looked like the connector was stuck.

Fix

  • portal.js: the dialog always asks for the estimate, as the Gmail dialog does. Before the sign-in the server answers without counts (liveSample returns null), so the choices show the cost per chat.
  • view.js: the note "Chat counts appear once EzyChat is connected" depends on the connection, not on the dialog mode, so the Reconnect dialog no longer shows it.
  • The dialog title is Reconnect EzyChat when EzyChat is already connected.
  • Version 0.1.8 and changelog entry.

Tests

test/ezychat.test.ts: per-chat figures and the note before the sign-in; no "once connected" note on Reconnect. 381 pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01J4pmGiYS6KFAWVkaAmk27U

jazztong and others added 2 commits November 3, 2026 01:06
…nect title; release 0.1.8

The dialog asked for the estimate only when EzyChat was connected, so before
the sign-in the choices said "Counting chats..." for ever. It now asks like the
Gmail dialog; the server answers without counts until the sign-in.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01J4pmGiYS6KFAWVkaAmk27U
@sonarqubecloud

sonarqubecloud Bot commented Nov 2, 2026

Copy link
Copy Markdown

@jazztong
jazztong merged commit d7c93c5 into main Nov 2, 2026
4 checks passed
@jazztong
jazztong deleted the fix/ezychat-dialog branch November 2, 2026 17:12
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