{{ message }}
Repository navigation
Profile と Settings のポップアップを追加 - #7
Merged
Merged
Conversation
/sparkle のメニューの Settings から、アイコン・名前・所属・Social Links・パスワードを変更できるようにする。Username は表示のみで変更不可。入力欄は ProfileFields に切り出して、新規登録フォームと共有する。パスワード変更用に POST /api/password を追加。現在のパスワードの打ち間違いでログアウトされないよう、送信前にブラウザで照合する。
メニューの Settings はログイン・新規登録と同じく Header のポップアップで開き、/settings ページは削除する。プロフィールを保存したら profile:updated イベントで /sparkle の名刺を読み直す。
メニューの Profile でアイコン・名前・所属・Social Links を編集する ProfileForm を、Settings でパスワードを変更する SettingsForm を開く。どちらも Header のポップアップで表示する。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

/sparkleのメニューから、ログインや新規登録と同じポップアップでプロフィール編集とパスワード変更をできるようにした。ページは遷移しない。#6 の上に積んでいる (401 のときにログイン画面へ戻す
AuthErrorを使うため)。#6 をマージすると、この PR のマージ先は自動でmainに変わる。メニューと中身
ProfileFormdisabled)SettingsForm実装
ProfileFields.tsxに切り出し、新規登録フォーム・ProfileForm・SettingsFormで共有する。新規登録フォームの見た目と動きは変えていないHeaderのポップアップにprofileとsettingsを追加し、AccountMenuから開く (スキャナー画面のメニューからも開ける)profile:updatedイベントを出し、/sparkleが名刺を読み直す/api/changeと/api/iconで保存するPOST /api/passwordを追加。現在の値で認証してから保存値を置き換え、ログインと同じレート制限をかけるdocs/password-hardening.mdの「パスワードの変更機能はまだ無い」を更新確認 (ローカルの
wrangler dev)/sparkleのメニューの Profile で、URL はそのままでポップアップが開き、現在の値が入っている。Username は編集できず、パスワード欄はない/sparkleに移動できる