-
Notifications
You must be signed in to change notification settings - Fork 39
多要素認証ページの画像更新 + リファクタリング #1836
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.
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
base: master
Are you sure you want to change the base?
多要素認証ページの画像更新 + リファクタリング #1836
Conversation
|
🚀 Deployed on https://deploy-preview-1836--utelecon.netlify.app |
seieric
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
以下のページについて、修正部分の文章と画像が整合性のとれる形で配置されていることを確認しました。 具体的には、説明の文章とそれに対する画像の内容が一致していることをチェックしました。
/utokyo_account/mfa/change//utokyo_account/mfa/fido-security_key//utokyo_account/mfa/initial//utokyo_account/mfa/reregister//utokyo_account/mfa/yubikey-totp/
また、src/components/ja/mfa以下の画像ファイルについて未使用の画像が残されていないことの確認として、少なくとも各画像が1つ以上のファイルから参照されていることを確認しました。
なお、説明されているアカウント設定時の画面遷移やアプリの挙動そのものについては確認していません。
多要素認証のページ全体の手入れです、主な内容は以下の通りです
手順の修正
画像の共通化と
<ArrowOverlay>コンポーネントの使用<ArrowOverlay>コンポーネントを使ってスクリーンショットに矢印を付けることで、複数の本人確認方法の説明で同じ画像を使い回せるようにしました<AltAddMethod>コンポーネントの幅広い使用<AltAddMethod>コンポーネントを、できるだけ多くの場面で使うようにしました認証アプリを使う手順のコンポーネント化
<ShowQr>コンポーネントに集約しました他の本人確認方法を使う手順のコンポーネント化
<SigninAnotherMethod>コンポーネントに集約しましたTODO(このPRのマージ後に行うこと)