CodeForPhilly / CodeForPhilly/codeforphilly-ng

Browser-validate the account-claim flow end-to-end

オープン
#48 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
TypeScript
スター
1
フォーク
1
平均マージ
5日 3時間
マージ済み PR(30日)
9

説明

## Context

The account-claim plan ([PR #46](https://github.com/CodeForPhilly/codeforphilly-ng/pull/46)) covers all paths via unit + integration tests against the in-process Fastify instance. Two validation criteria depend on the running browser experience and were not flipped at merge:

- OAuth callback with candidates → claim screen renders the candidate(s) with the right info
- Multi-candidate picker works; selecting one claims it; others remain unclaimed

The implementation is complete; this is purely a manual / playwright verification gap. The parent-repo dev server was busy during the implementation window so I couldn't drive a real GitHub OAuth flow through the new screens without disrupting other agents.

## Verification

Run `dev` in this branch (or a checked-out post-merge `main`) and:

1. Sign in via GitHub against a laddr-era email that matches a single legacy Person. Confirm the claim screen renders one card with `matchedVia: email`, fullName, slug, and member count.
2. Repeat against a GH identity whose `gh.login` and email both match different legacy Persons (multi-candidate). Confirm the picker shows both cards; select one; confirm only that Person gets the GH identity and the other stays unclaimed.

Notes: a quick way to seed the multi-candidate path locally is to add two `people/*.toml` records with distinct slugs and email-match one to your real `primary` GH email, and have the GH `login` field equal the other slug.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

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

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