Skip to content

メールアドレス・パスワードでのログインに対応 - #6

Open
shirafukayayoi wants to merge 2 commits into
MidnightTale:masterfrom
shirafukayayoi:codex/email-password-login
Open

shirafukayayoi wants to merge 2 commits into
MidnightTale:masterfrom
shirafukayayoi:codex/email-password-login

Conversation

@shirafukayayoi

Copy link
Copy Markdown

QRログインに加えて、メールアドレスとパスワードでログインできるようにしました。スマホで6桁の認証番号を確認し、追加のPCセッションとして接続します。

変更内容

  • メール入力画面、認証番号の表示、再入力、QRへの切り替えを追加しました。切り替え時は進行中の認証をキャンセルします。
  • パスワードは保存せず、送信・キャンセル・失敗時に入力欄から消去します。ライブラリの例外は入力値を含まない表示用エラーへ変換します。
  • DESKTOPWIN のセッションを保存・復元し、スマホの認証待ちに合わせてログイン要求の待機時間を延長しました。
  • PCセッションでは実験的な通話機能を無効にし、QRログインが必要な旨を表示します。
  • @evex/linejs と型定義を3.5.0に固定しました。

日本語UI・フォント対応は #4、本文の復号とグループ名の修正は #5 で分けています。このPRは単独でもビルドでき、3本を組み合わせても競合なく結合できることを確認しました。

確認

  • 単独でRust 10件・Deno 10件のテストが成功しました。
  • Rustのフォーマット確認、警告をエラーにしたClippy、リリースビルド、sidecarのコンパイルと未認証起動を確認しました。
  • 実際のLINEアカウントへ接続しないモックの画面テストで、入力検証、認証番号の表示、失敗後の再入力、認証のキャンセル、ログイン、再起動後の復元、ログアウトを確認しました。
  • モックの保存データとログにパスワードが残らないことを確認しました。
  • 日本語UIと日本語向けフォント設定に対応 #4・#5との結合状態でRust 15件・Deno 17件とClippy、フォーマット確認が成功しました。

今回の切り出したブランチでは、実サービスへの新規ログインは実施していません。

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