Skip to content

Recover Captain sign-in after desktop reinstall - #225

Merged
sridharkalaibala merged 2 commits into
mainfrom
fix/reinstalled-till-reconnect
Oct 30, 2026
Merged

sridharkalaibala merged 2 commits into
mainfrom
fix/reinstalled-till-reconnect

Conversation

@sridharkalaibala

@sridharkalaibala sridharkalaibala commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Captain could become stuck after a till reinstall because a missing paired session was reported as an offline server. Rejected or expired password-only logins also lacked an in-place recovery prompt, and reconnecting could retain obsolete pairing secrets.

Offer sign-in at the saved issuing address, retain saved orders and addresses, and replace old credentials only after validating the same shop, staff member and branch. Keep revoked-device and unverified-route protections intact. A successful recovery permits another prompt after a later reinstall.

Validation completed locally:

  • 692 unit tests and all 1,043 browser tests passed (npm run check).
  • Browser coverage includes paired-session loss, rejected and expired password-only login, Wi-Fi search, route switching and saved-work isolation.
  • Production web build, Android release compilation/signing, attribution and diff checks passed. The final testing APK will be rebuilt from merged main.
  • Physical desktop uninstall/reinstall with a real phone has not been verified.

Sync: existing secure local credentials only. No business schema, Gateway or sync-agent changes; session secrets remain local. Existing POS APIs suffice. Rollout requires the updated Captain APK.

The earlier disk-space blocker has been resolved and the complete local gate has now passed.

Published signed testing APK: https://github.com/Posnic/captain/releases/tag/captain-1.3.72-test.1 . Built from merged main 573770a, version 1.3.72 (10372); signing certificate matches 1.3.71. Uploaded APK digest matches the local SHA-256.

@sridharkalaibala
sridharkalaibala marked this pull request as ready for review October 30, 2026 20:33
@sridharkalaibala
sridharkalaibala merged commit 573770a into main Oct 30, 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.

1 participant