The "You're in." confirmation doesn't say which email address it used

クローズ 初心者向け
  • 何か問題がありますか？
#715 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
85/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
next.js, react, typescript
領域
frontend

調査の方向性

src/app/s/[slug]/commit-dialog.tsx を開き、success ブランチを特定する;success.emailed が true のとき、メール送信済みのアドレスを最初の行に挿入する。src/app/s/[slug]/signup-view.test.tsx を更新して両方の文言ケースを固定する。pnpm lint && pnpm typecheck && pnpm knip && pnpm test を実行する;サインアップ確認がアドレスを表示し、すべてのチェックが通れば完了。

索引モデルが issue の本文から書いたものです。

説明

enhancement pipeline:ready pipeline:ux

Needs your OK: this changes what people read after they sign up. Swap pipeline:proposed for pipeline:ready to have it built.

What happens

After someone signs up with an email, the "You're in." sheet (the success branch in src/app/s/[slug]/commit-dialog.tsx) says "We've saved your spot for . Bookmark this link to edit or cancel later:". It doesn't name the address it used, or say an email went out. The address first appears on the edit page ("You're editing this as …").

So a typo the suggestion list doesn't know goes unnoticed. suggestEmail (src/lib/email-suggest.ts) only checks a list of common providers: [email protected] gets "Did you mean …gmail.com?", but [email protected], or a typo in a school's own domain, gets nothing. That person gets no confirmation and no reminders, and nothing tells them why. Their browser also remembers the address, so the form fills in the same typo next time they sign up.

Fix, in outline

  • When success.emailed is true, name the address in the first line, before the link.
  • Leave the branch for an optional email left blank as it is.

Choices this makes

  • Wording. Default: "We've saved your spot for and emailed this link to . Bookmark it to edit or cancel later:". It says what happened and where, in one sentence.
  • No "wrong address?" action. Default: name the address only. The edit page can't change an email today, so a "change it" link would need that first. Someone who spots a typo can cancel and sign up again.

Watch for

  • src/app/s/[slug]/signup-view.test.tsx pins the "You're in." text.
  • Long addresses at 390px: the line must wrap, not scroll sideways.
  • CHANGELOG entry.

Done when

  • Signing up with an email shows that address in "You're in."; with an optional email left blank, the text is unchanged.
  • A test in signup-view.test.tsx pins both.
  • pnpm lint && pnpm typecheck && pnpm knip && pnpm test pass.

Found by the UX walkthrough, scenario P1, on opensignup-prod main at 1234209 (this repo at dc99921). Checked against commit-dialog.tsx and email-suggest.ts at dc99921.

Generated by Claude Code

主要言語
TypeScript
スター
19
フォーク
4
平均マージ
19時間 15分
マージ済み PR（30日）
205

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

richshaw/OpenSignup のほかの issue

richshaw/OpenSignup の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。