Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: 登録時のフォームにおいて送信できる条件を修正 #10238

Merged
merged 2 commits into from
Mar 7, 2023
Merged

fix: 登録時のフォームにおいて送信できる条件を修正 #10238

merged 2 commits into from
Mar 7, 2023

Conversation

Ry0taK
Copy link
Contributor

@Ry0taK Ry0taK commented Mar 6, 2023 •
edited
Loading

What

  • アカウント作成時のフォームにおいてユーザー名の重複やメールアドレスの重複などが発生している場合にフォームの送信ボタンを無効にするように
  • パスワードの一致状況が match である場合にのみフォームを送信できるように

Why

  • 警告が出ているにも関わらず送信ボタンが押下出来る状態となっていることで、ユーザーの混乱を招くため
  • パスワードを入力していない状態の場合、パスワードの一致状況がnullとなってしまい、passwordRetypeState === 'not-match'がfalseになってしまっていたため。

Additional info (optional)

  • partially fixes 新規登録にメール認証が必須の時、ユーザー名が被っていてもメール送信まではできてしまう #10076
  • メール登録を必須にしているインスタンスにおいて、以下の状況で動作確認済み
    1. メールアドレスが既に使用されている場合、送信ボタンが無効になること
    2. ユーザー名が重複している場合、送信ボタンが無効になること
    3. 上記の両方を満たしている状態で送信ボタンが無効になること
    4. パスワードフィールドが空の場合に送信ボタンが無効になること
    5. 全てのフィールドが適切に記入されている場合に送信ボタンが有効になること
  • メール登録が必須でないインスタンスにおいて、以下の状況で動作確認済み
    1. ユーザー名が重複している場合、送信ボタンが無効になること
    2. パスワードフィールドが空の場合に送信ボタンが無効になること
    3. ユーザー名とパスワードが適切に記入されている場合に送信ボタンが有効になること

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Mar 6, 2023
@Ry0taK Ry0taK marked this pull request as ready for review March 6, 2023 10:55
@github-actions github-actions bot requested review from acid-chicken and syuilo March 6, 2023 10:55
@Ry0taK Ry0taK mentioned this pull request Mar 6, 2023
Merged
@syuilo
Copy link
Member

syuilo commented Mar 7, 2023

frontendのe2eテスト追加しておくと良さそうです

@Ry0taK
Copy link
Contributor Author

Ry0taK commented Mar 7, 2023

テスト追加してみました

@syuilo syuilo merged commit 6feead5 into misskey-dev:develop Mar 7, 2023
@syuilo
Copy link
Member

syuilo commented Mar 7, 2023

🙏

@saschanaz
Copy link
Member

この後テスト時々落ちるようになりましたが、何が起きているのか分かりにくいです🤔

@Ry0taK Ry0taK mentioned this pull request Mar 7, 2023
Closed
@Ry0taK
Copy link
Contributor Author

Ry0taK commented Mar 7, 2023

@saschanaz 原因がわかったのでIssueを切っておきました: #10262

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

@acid-chicken acid-chicken Awaiting requested review from acid-chicken

@syuilo syuilo Awaiting requested review from syuilo

Assignees
No one assigned
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Milestone
No milestone
3 participants
@Ry0taK @syuilo @saschanaz