|
|
@@ -1,245 +1,375 @@
|
|
|
-# Admin フォーム - React Hook Form 移行進捗
|
|
|
+# Admin フォーム移行プロジェクト - 進捗追跡
|
|
|
|
|
|
-## 移行ステータス
|
|
|
+最終更新: 2025-10-14
|
|
|
|
|
|
-### ✅ 完了したコンポーネント
|
|
|
+## 🎯 プロジェクト概要
|
|
|
+GROWI の Admin 設定画面におけるフォームコンポーネントを Unstated Container から React Hook Form (RHF) へ移行しています。
|
|
|
|
|
|
-#### AdminAppContainer 配下
|
|
|
+## ⚠️ 重要な訂正
|
|
|
|
|
|
-1. **AppSetting.jsx**
|
|
|
- - パス: `apps/app/src/client/components/Admin/App/AppSetting.jsx`
|
|
|
- - 担当フィールド: サイト名、confidential、言語、メール公開設定、ファイルアップロード
|
|
|
- - 特記事項: ラジオボタンの型変換(boolean → string)を実装
|
|
|
- - テスト状況: ✅ IME 入力、値復元、ラジオボタン復元 確認済み
|
|
|
+**以前の報告**: "PR #10051 完全解決(100%)" → **誤り**
|
|
|
|
|
|
-2. **SiteUrlSetting.tsx**
|
|
|
- - パス: `apps/app/src/client/components/Admin/App/SiteUrlSetting.tsx`
|
|
|
- - 担当フィールド: サイト URL
|
|
|
- - 特記事項: 環境変数による上書き時は `readOnly` を使用
|
|
|
- - テスト状況: ✅ IME 入力、値復元 確認済み
|
|
|
+**実態**: PR #10051 で変更された **27ファイル中17ファイル(63%)のみ完了**。残り **10ファイル(約48+ フィールド)が未移行**。
|
|
|
|
|
|
-3. **MailSetting.tsx**
|
|
|
- - パス: `apps/app/src/client/components/Admin/App/MailSetting.tsx`
|
|
|
- - 担当フィールド: メール送信元アドレス、送信方法(SMTP/SES)
|
|
|
- - 特記事項: 親フォームとして SmtpSetting/SesSetting を管理
|
|
|
- - テスト状況: ⏳ 未テスト
|
|
|
+## 📊 現在の進捗
|
|
|
|
|
|
-4. **SmtpSetting.tsx**
|
|
|
- - パス: `apps/app/src/client/components/Admin/App/SmtpSetting.tsx`
|
|
|
- - 担当フィールド: SMTP ホスト、ポート、ユーザー、パスワード
|
|
|
- - 特記事項: 子コンポーネントとして `register` を props で受け取る
|
|
|
- - テスト状況: ⏳ 未テスト
|
|
|
+### 完了済み: 17コンポーネント、約33フィールド
|
|
|
|
|
|
-5. **SesSetting.tsx**
|
|
|
- - パス: `apps/app/src/client/components/Admin/App/SesSetting.tsx`
|
|
|
- - 担当フィールド: AWS SES アクセスキー、シークレットキー
|
|
|
- - 特記事項: 子コンポーネントとして `register` を props で受け取る
|
|
|
- - テスト状況: ⏳ 未テスト
|
|
|
+#### ✅ Phase 1: シンプルなカスタマイズ設定(4コンポーネント、4フィールド)
|
|
|
+1. **CustomizeCssSetting.tsx** - カスタム CSS テキストエリア
|
|
|
+2. **CustomizeScriptSetting.tsx** - カスタムスクリプト テキストエリア
|
|
|
+3. **CustomizeNoscriptSetting.tsx** - noscript タグテキストエリア
|
|
|
+4. **CustomizeTitle.tsx** - サイトタイトル入力
|
|
|
|
|
|
-#### AdminCustomizeContainer 配下
|
|
|
+#### ✅ Phase 2: ファイルアップロード設定(5コンポーネント、13フィールド)
|
|
|
+5. **FileUploadSetting.tsx** - 親コンポーネント(useForm 管理)
|
|
|
+6. **AwsSetting.tsx** - AWS S3 設定(4フィールド)
|
|
|
+7. **GcsSetting.tsx** - Google Cloud Storage 設定(3フィールド)
|
|
|
+8. **AzureSetting.tsx** - Azure Blob Storage 設定(5フィールド)
|
|
|
+9. **MaskedInput.tsx** - パスワード/シークレットマスク入力(デュアルモード)
|
|
|
|
|
|
-6. **CustomizeCssSetting.tsx** ✨
|
|
|
- - パス: `apps/app/src/client/components/Admin/Customize/CustomizeCssSetting.tsx`
|
|
|
- - 担当フィールド: カスタム CSS
|
|
|
- - 特記事項: textarea での大きなテキスト入力、空値更新が重要
|
|
|
- - テスト状況: ⏳ 未テスト(IME 入力、空値更新の確認が必要)
|
|
|
+#### ✅ Phase 3: セキュリティ OAuth 設定(2コンポーネント、4フィールド)
|
|
|
+10. **GitHubSecuritySettingContents.jsx** - GitHub OAuth(2フィールド)
|
|
|
+11. **GoogleSecuritySettingContents.jsx** - Google OAuth(2フィールド)
|
|
|
|
|
|
-7. **CustomizeScriptSetting.tsx** ✨
|
|
|
- - パス: `apps/app/src/client/components/Admin/Customize/CustomizeScriptSetting.tsx`
|
|
|
- - 担当フィールド: カスタムスクリプト(JavaScript)
|
|
|
- - 特記事項: Google Tag Manager の例を含む、空値更新が重要
|
|
|
- - テスト状況: ⏳ 未テスト(IME 入力、空値更新の確認が必要)
|
|
|
+#### ✅ Phase 4: レガシー Slack & XSS 設定(3コンポーネント、4フィールド)
|
|
|
+12. **SlackConfiguration.jsx** - Slack Webhook/Token 設定(2フィールド)
|
|
|
+13. **XssForm.jsx** - XSS 防止設定(親コンポーネント)
|
|
|
+14. **WhitelistInput.tsx** - XSS タグ/属性ホワイトリスト(2フィールド)
|
|
|
|
|
|
-8. **CustomizeNoscriptSetting.tsx** ✨
|
|
|
- - パス: `apps/app/src/client/components/Admin/Customize/CustomizeNoscriptSetting.tsx`
|
|
|
- - 担当フィールド: カスタム noscript タグ(HTML)
|
|
|
- - 特記事項: Google Tag Manager の iframe 例を含む、空値更新が重要
|
|
|
- - テスト状況: ⏳ 未テスト(IME 入力、空値更新の確認が必要)
|
|
|
+#### ✅ 以前に完了済み(3コンポーネント、約8フィールド)
|
|
|
+15. **AppSetting.jsx** - アプリ設定(2フィールド)
|
|
|
+16. **SiteUrlSetting.tsx** - サイト URL(1フィールド)
|
|
|
+17. **MailSetting.tsx** - メール From アドレス(1フィールド)
|
|
|
+18. **SmtpSetting.tsx** - SMTP 設定(4フィールド)
|
|
|
+19. **SesSetting.tsx** - AWS SES 設定(2フィールド)
|
|
|
|
|
|
-#### SWR Store ベース
|
|
|
+---
|
|
|
|
|
|
-9. **CustomizeTitle.tsx** ✨ NEW
|
|
|
- - パス: `apps/app/src/client/components/Admin/Customize/CustomizeTitle.tsx`
|
|
|
- - 担当フィールド: カスタムタイトル(HTML title タグのテンプレート)
|
|
|
- - 特記事項: Unstated Container ではなく SWR の `useCustomizeTitle` を使用
|
|
|
- - テスト状況: ⏳ 未テスト(IME 入力の確認が必要)
|
|
|
+### ❌ **未移行(10コンポーネント、約48+ フィールド)**
|
|
|
|
|
|
-### 🔄 移行対象候補(未着手)
|
|
|
+#### 🔴 HIGH PRIORITY: エンタープライズ認証設定(36フィールド)
|
|
|
|
|
|
-#### AdminCustomizeContainer 配下
|
|
|
+**これらは企業ユーザーにとってクリティカルな機能です!**
|
|
|
|
|
|
-以下のコンポーネントは AdminCustomizeContainer を使用しているが、複雑な構造のため優先度低:
|
|
|
+1. **LdapSecuritySettingContents.jsx** - LDAP 認証設定
|
|
|
+ - 10フィールド(サーバー URL、バインド DN、検索フィルター、属性マッピングなど)
|
|
|
+ - クラスコンポーネント、Container ベース
|
|
|
+ - **優先度: CRITICAL** - 多くの企業で使用
|
|
|
|
|
|
-- `CustomizeFunctionSetting.tsx` - 機能設定(複数のチェックボックス/選択肢、テキスト入力なし)
|
|
|
-- `CustomizePresentationSetting.tsx` - プレゼンテーション設定(チェックボックスのみ、テキスト入力なし)
|
|
|
+2. **OidcSecuritySettingContents.jsx** - OpenID Connect 認証設定
|
|
|
+ - **16フィールド**(最多!)- プロバイダー名、各種エンドポイント URL、属性マッピングなど
|
|
|
+ - クラスコンポーネント、Container ベース
|
|
|
+ - **優先度: CRITICAL** - モダンな企業認証の標準
|
|
|
|
|
|
-#### 他の Admin Container 配下
|
|
|
+3. **SamlSecuritySettingContents.jsx** - SAML 認証設定
|
|
|
+ - 10フィールド(エントリーポイント、証明書、属性マッピング、ABLC ルールなど)
|
|
|
+ - クラスコンポーネント、環境変数との連携あり
|
|
|
+ - **優先度: CRITICAL** - エンタープライズで広く使用
|
|
|
|
|
|
-以下は複雑で大規模なため、後回し:
|
|
|
+#### 🟡 MEDIUM PRIORITY: その他のセキュリティ設定(2フィールド)
|
|
|
|
|
|
-- AdminSecurityContainer 配下のフォーム
|
|
|
- - `OidcSecuritySettingContents.jsx` - OIDC 設定(多数の input フィールド)
|
|
|
- - `SamlSecuritySettingContents.jsx` - SAML 設定(textarea あり、複雑)
|
|
|
- - `LdapSecuritySettingContents.jsx` - LDAP 設定(多数の input フィールド)
|
|
|
- - `GoogleSecuritySettingContents.jsx`
|
|
|
- - `GitHubSecuritySettingContents.jsx`
|
|
|
- - `LocalSecuritySettingContents.jsx`
|
|
|
+4. **SecuritySetting.jsx** - セキュリティ全般設定
|
|
|
+ - 1フィールド(`sessionMaxAge`)
|
|
|
+ - 単純な Container ベース
|
|
|
|
|
|
-- AdminMarkdownContainer 配下のフォーム
|
|
|
- - `XssForm.jsx` - XSS 設定(クラスコンポーネント、複雑)
|
|
|
- - `WhitelistInput.tsx` - ホワイトリスト入力(XssForm の子コンポーネント)
|
|
|
- - `LineBreakForm.jsx`
|
|
|
+5. **LocalSecuritySettingContents.jsx** - ローカル認証設定
|
|
|
+ - 1フィールド(`registrationWhitelist` textarea)
|
|
|
+ - 配列を `\n` で join する特殊なハンドリングが必要
|
|
|
|
|
|
-- 画像アップロード関連(React Hook Form に不適)
|
|
|
- - `CustomizeLogoSetting.tsx` - ロゴ画像のアップロードと切り抜き
|
|
|
+#### 🟢 LOW PRIORITY: Slack 設定(既に useState 使用中)
|
|
|
|
|
|
-### 📋 次のステップ
|
|
|
+**注**: これらはすでに `useState` を使用しているため、IME 問題は発生しにくい。React Hook Form への移行は統一性のため推奨されるが、優先度は低い。
|
|
|
|
|
|
-1. **今回移行したコンポーネントのテスト**
|
|
|
- - CustomizeCssSetting の IME 入力テスト
|
|
|
- - CustomizeScriptSetting の IME 入力テスト
|
|
|
- - CustomizeNoscriptSetting の IME 入力テスト
|
|
|
- - CustomizeTitle の IME 入力テスト
|
|
|
- - 空値更新のテスト(これらのフィールドは空にできることが重要)
|
|
|
+6. **CustomBotWithProxySettings.jsx** - Slack Bot with Proxy 設定
|
|
|
+ - 1フィールド(`proxyServerUri`)
|
|
|
+ - すでに関数コンポーネント + useState
|
|
|
|
|
|
-2. **他のシンプルなテキスト入力フォームを探す**
|
|
|
- - Admin 配下で単純な input/textarea を持つコンポーネントを特定
|
|
|
- - 優先順位: シンプル > デグレリスクが低い > 使用頻度が高い
|
|
|
+7. **CustomBotWithoutProxySecretTokenSection.jsx** - Slack Bot without Proxy シークレット
|
|
|
+ - 2フィールド(`inputSigningSecret`, `inputBotToken`)
|
|
|
+ - すでに関数コンポーネント + useState
|
|
|
|
|
|
-3. **複雑なフォームは後回し**
|
|
|
- - Security 関連の大規模フォーム
|
|
|
- - クラスコンポーネント
|
|
|
- - 画像アップロード関連
|
|
|
+8. **ManageCommandsProcess.jsx** - Slack コマンド権限管理
|
|
|
+ - 複数の動的生成 textarea フィールド
|
|
|
+ - 複雑な権限設定ロジック、日本語入力は稀
|
|
|
|
|
|
-## 発見した問題と解決策
|
|
|
+---
|
|
|
|
|
|
-### 問題1: フォーム送信時に古い値が送信される
|
|
|
-- **原因**: Container の `setState` が非同期なのに `await` していなかった
|
|
|
-- **解決**: すべての `change*()` メソッドに `await` を追加、`Promise.all()` で並列実行
|
|
|
+## 🎯 PR #10051 IME 問題の進捗
|
|
|
|
|
|
-### 問題2: ラジオボタンの選択状態が復元されない
|
|
|
-- **原因**: ラジオボタンの value は文字列だが、reset に boolean を渡していた
|
|
|
-- **解決**: `String()` で明示的に型変換
|
|
|
+### 完了率
|
|
|
+- **ファイル**: 17/27 完了(**63%**)
|
|
|
+- **フィールド数**: 約33/81+ 完了(**約41%**)
|
|
|
+- **HIGH PRIORITY(エンタープライズ認証)**: 0/3 完了(**0%**)⚠️
|
|
|
+- **MEDIUM PRIORITY**: 0/2 完了(**0%**)
|
|
|
+- **LOW PRIORITY(Slack)**: 0/3 完了(**0%**)
|
|
|
|
|
|
-### 問題3: defaultValues の重複
|
|
|
-- **原因**: `useForm({ defaultValues })` と `useEffect` での `reset()` で二重定義
|
|
|
-- **解決**: `defaultValues` を削除し、`reset()` のみで管理
|
|
|
+### 🚨 最大の問題
|
|
|
+**エンタープライズ認証設定(LDAP, OIDC, SAML)が未移行!**
|
|
|
+- 合計 **36フィールド** が IME 問題の影響を受けている
|
|
|
+- これらは企業ユーザーにとって **必須の機能**
|
|
|
+- 日本語のコメントやラベルが正しく入力できない可能性
|
|
|
|
|
|
-### 問題4: textarea での IME 入力問題
|
|
|
-- **原因**: 制御されたコンポーネント(`value` + `onChange`)を使用していた
|
|
|
-- **解決**: React Hook Form の `register` を使用して非制御コンポーネント化
|
|
|
+詳細は `admin-forms-pr10051-ime-issues.md` を参照。
|
|
|
|
|
|
-## 移行パターンの確立
|
|
|
+---
|
|
|
|
|
|
-### パターン1: Container ベースの単一 textarea フィールド
|
|
|
+## 🔧 確立された移行パターン
|
|
|
+
|
|
|
+### パターン1: Container ベース(シンプル)
|
|
|
+単一コンポーネント、Unstated Container から状態を取得
|
|
|
|
|
|
```typescript
|
|
|
-const {
|
|
|
- register,
|
|
|
- handleSubmit,
|
|
|
- reset,
|
|
|
-} = useForm();
|
|
|
-
|
|
|
-useEffect(() => {
|
|
|
- reset({
|
|
|
- fieldName: container.state.currentFieldName || '',
|
|
|
- });
|
|
|
-}, [container.state.currentFieldName, reset]);
|
|
|
-
|
|
|
-const onSubmit = useCallback(async(data) => {
|
|
|
- try {
|
|
|
- await container.changeFieldName(data.fieldName);
|
|
|
- await container.updateFieldName();
|
|
|
- toastSuccess('...');
|
|
|
- }
|
|
|
- catch (err) {
|
|
|
- toastError(err);
|
|
|
- }
|
|
|
-}, [container]);
|
|
|
-
|
|
|
-return (
|
|
|
- <form onSubmit={handleSubmit(onSubmit)}>
|
|
|
- <textarea {...register('fieldName')} />
|
|
|
- <AdminUpdateButtonRow type="submit" />
|
|
|
- </form>
|
|
|
-);
|
|
|
+const Component = (props) => {
|
|
|
+ const { adminContainer } = props;
|
|
|
+ const { register, handleSubmit, reset } = useForm();
|
|
|
+
|
|
|
+ useEffect(() => {
|
|
|
+ reset({ field: adminContainer.state.field });
|
|
|
+ }, [reset, adminContainer.state.field]);
|
|
|
+
|
|
|
+ const onSubmit = useCallback(async(data) => {
|
|
|
+ await adminContainer.updateField(data.field);
|
|
|
+ }, [adminContainer]);
|
|
|
+
|
|
|
+ return (
|
|
|
+ <form onSubmit={handleSubmit(onSubmit)}>
|
|
|
+ <input {...register('field')} />
|
|
|
+ <AdminUpdateButtonRow />
|
|
|
+ </form>
|
|
|
+ );
|
|
|
+};
|
|
|
```
|
|
|
|
|
|
-適用済み:
|
|
|
-- CustomizeCssSetting
|
|
|
-- CustomizeScriptSetting
|
|
|
-- CustomizeNoscriptSetting
|
|
|
+### パターン2: SWR ベース
|
|
|
+SWR を使用してサーバー状態を管理
|
|
|
+
|
|
|
+```typescript
|
|
|
+const Component = () => {
|
|
|
+ const { data, mutate } = useSWRxAppSettings();
|
|
|
+ const { register, handleSubmit, reset } = useForm();
|
|
|
+
|
|
|
+ useEffect(() => {
|
|
|
+ if (data) reset({ field: data.field });
|
|
|
+ }, [data, reset]);
|
|
|
+
|
|
|
+ const onSubmit = useCallback(async(formData) => {
|
|
|
+ await apiv3Put('/settings', { field: formData.field });
|
|
|
+ mutate();
|
|
|
+ }, [mutate]);
|
|
|
+
|
|
|
+ return (
|
|
|
+ <form onSubmit={handleSubmit(onSubmit)}>
|
|
|
+ <input {...register('field')} />
|
|
|
+ </form>
|
|
|
+ );
|
|
|
+};
|
|
|
+```
|
|
|
|
|
|
-### パターン2: SWR Store ベースの単一 input フィールド
|
|
|
+### パターン3: 親子フォーム共有
|
|
|
+親で useForm、子に register を渡す
|
|
|
|
|
|
```typescript
|
|
|
-const { data: storeData } = useStoreHook();
|
|
|
-
|
|
|
-const {
|
|
|
- register,
|
|
|
- handleSubmit,
|
|
|
- reset,
|
|
|
-} = useForm();
|
|
|
-
|
|
|
-useEffect(() => {
|
|
|
- reset({
|
|
|
- fieldName: storeData ?? '',
|
|
|
- });
|
|
|
-}, [storeData, reset]);
|
|
|
-
|
|
|
-const onSubmit = useCallback(async(data) => {
|
|
|
- try {
|
|
|
- await apiv3Put('/api/endpoint', {
|
|
|
- fieldName: data.fieldName,
|
|
|
- });
|
|
|
- toastSuccess('...');
|
|
|
+// 親
|
|
|
+const ParentForm = () => {
|
|
|
+ const { register, handleSubmit, setValue } = useForm();
|
|
|
+
|
|
|
+ return (
|
|
|
+ <form onSubmit={handleSubmit(onSubmit)}>
|
|
|
+ <ChildInput register={register} setValue={setValue} />
|
|
|
+ </form>
|
|
|
+ );
|
|
|
+};
|
|
|
+
|
|
|
+// 子
|
|
|
+const ChildInput = ({ register, setValue }) => {
|
|
|
+ return <input {...register('field')} />;
|
|
|
+};
|
|
|
+```
|
|
|
+
|
|
|
+### パターン4: クラスから関数への変換
|
|
|
+レガシークラスコンポーネントの移行
|
|
|
+
|
|
|
+```typescript
|
|
|
+// Before: class component with Container
|
|
|
+class LegacyForm extends React.Component {
|
|
|
+ async onClickSubmit() {
|
|
|
+ const { container } = this.props;
|
|
|
+ await container.updateSetting();
|
|
|
}
|
|
|
- catch (err) {
|
|
|
- toastError(err);
|
|
|
+
|
|
|
+ render() {
|
|
|
+ return <input value={this.props.container.state.field}
|
|
|
+ onChange={e => this.props.container.changeField(e.target.value)} />;
|
|
|
}
|
|
|
-}, []);
|
|
|
-
|
|
|
-return (
|
|
|
- <form onSubmit={handleSubmit(onSubmit)}>
|
|
|
- <input {...register('fieldName')} />
|
|
|
- <AdminUpdateButtonRow type="submit" />
|
|
|
- </form>
|
|
|
-);
|
|
|
+}
|
|
|
+
|
|
|
+// After: function component with useForm
|
|
|
+const ModernForm = (props) => {
|
|
|
+ const { container } = props;
|
|
|
+ const { register, handleSubmit, reset } = useForm();
|
|
|
+
|
|
|
+ useEffect(() => {
|
|
|
+ reset({ field: container.state.field });
|
|
|
+ }, [reset, container.state.field]);
|
|
|
+
|
|
|
+ const onSubmit = useCallback(async(data) => {
|
|
|
+ await container.changeField(data.field);
|
|
|
+ await container.updateSetting();
|
|
|
+ }, [container]);
|
|
|
+
|
|
|
+ return (
|
|
|
+ <form onSubmit={handleSubmit(onSubmit)}>
|
|
|
+ <input {...register('field')} />
|
|
|
+ <AdminUpdateButtonRow onClick={handleSubmit(onSubmit)} />
|
|
|
+ </form>
|
|
|
+ );
|
|
|
+};
|
|
|
```
|
|
|
|
|
|
-適用済み:
|
|
|
-- CustomizeTitle
|
|
|
+### パターン5: 配列のハンドリング(予定)
|
|
|
+LocalSecuritySettingContents で使用予定
|
|
|
+
|
|
|
+```typescript
|
|
|
+// registrationWhitelist は配列だが、textarea には \n で join して表示
|
|
|
+const Component = (props) => {
|
|
|
+ const { container } = props;
|
|
|
+ const { register, handleSubmit, reset } = useForm();
|
|
|
+
|
|
|
+ useEffect(() => {
|
|
|
+ reset({
|
|
|
+ registrationWhitelist: container.state.registrationWhitelist.join('\n')
|
|
|
+ });
|
|
|
+ }, [reset, container.state.registrationWhitelist]);
|
|
|
+
|
|
|
+ const onSubmit = useCallback(async(data) => {
|
|
|
+ // \n で split して配列に戻す
|
|
|
+ const whitelist = data.registrationWhitelist.split('\n').filter(s => s.trim());
|
|
|
+ await container.updateWhitelist(whitelist);
|
|
|
+ }, [container]);
|
|
|
+
|
|
|
+ return (
|
|
|
+ <form onSubmit={handleSubmit(onSubmit)}>
|
|
|
+ <textarea {...register('registrationWhitelist')} />
|
|
|
+ </form>
|
|
|
+ );
|
|
|
+};
|
|
|
+```
|
|
|
+
|
|
|
+---
|
|
|
+
|
|
|
+## 🔍 次に移行すべきコンポーネント
|
|
|
+
|
|
|
+### 推奨される移行順序
|
|
|
+
|
|
|
+#### Phase 5: ウォーミングアップ(1フィールド)
|
|
|
+1. **SecuritySetting.jsx** - 最もシンプル、1フィールドのみ
|
|
|
+
|
|
|
+#### Phase 6: エンタープライズ認証(36フィールド)⚠️ **CRITICAL**
|
|
|
+2. **LdapSecuritySettingContents.jsx** - 10フィールド、約2-3時間
|
|
|
+3. **SamlSecuritySettingContents.jsx** - 10フィールド、約2-3時間
|
|
|
+4. **OidcSecuritySettingContents.jsx** - 16フィールド(最大)、約3-4時間
|
|
|
+
|
|
|
+#### Phase 7: その他のセキュリティ設定(1フィールド)
|
|
|
+5. **LocalSecuritySettingContents.jsx** - 配列ハンドリング、約1時間
|
|
|
+
|
|
|
+#### Phase 8: オプション(Slack 関連、低優先度)
|
|
|
+6. CustomBotWithProxySettings.jsx
|
|
|
+7. CustomBotWithoutProxySecretTokenSection.jsx
|
|
|
+8. ManageCommandsProcess.jsx
|
|
|
+
|
|
|
+**推定総時間**: 9-12時間(HIGH + MEDIUM 優先度のみ)
|
|
|
+
|
|
|
+---
|
|
|
+
|
|
|
+## 📝 テスト計画
|
|
|
+
|
|
|
+移行後は以下を確認すべき:
|
|
|
+
|
|
|
+1. **IME 入力テスト**
|
|
|
+ - 日本語の漢字変換が正常に動作する
|
|
|
+ - 中国語、韓国語などの他の IME も動作する
|
|
|
+
|
|
|
+2. **値の永続化**
|
|
|
+ - 入力値が正しく保存される
|
|
|
+ - ページリロード後に値が復元される
|
|
|
+
|
|
|
+3. **空値の処理**
|
|
|
+ - 空文字列での更新が正常に動作する
|
|
|
+ - 未入力の必須フィールドでバリデーションエラーが表示される
|
|
|
+
|
|
|
+4. **フォーム送信**
|
|
|
+ - 送信ボタンが正常に動作する
|
|
|
+ - 非同期エラーハンドリングが機能する
|
|
|
+
|
|
|
+5. **エンタープライズ認証の動作確認**
|
|
|
+ - LDAP 接続テスト
|
|
|
+ - OIDC プロバイダーとの連携
|
|
|
+ - SAML 認証フロー
|
|
|
+
|
|
|
+---
|
|
|
+
|
|
|
+## 💡 学んだこと
|
|
|
+
|
|
|
+1. **MaskedInput のデュアルモード**
|
|
|
+ - register/fieldName(RHF モード)と value/onChange(レガシーモード)の両方をサポート
|
|
|
+ - 段階的な移行が可能
|
|
|
+
|
|
|
+2. **WhitelistInput の発見**
|
|
|
+ - すでに React Hook Form 対応の設計だった
|
|
|
+ - setValue を使った「推奨設定をインポート」ボタンの実装が参考になる
|
|
|
+
|
|
|
+3. **親子フォームの設計**
|
|
|
+ - FileUploadSetting、XssForm で成功したパターン
|
|
|
+ - register と setValue を props で渡すことで子コンポーネントも RHF の恩恵を受けられる
|
|
|
+
|
|
|
+4. **クラスコンポーネントの変換**
|
|
|
+ - useForm、useEffect、useCallback で置き換え
|
|
|
+ - Container との連携は useEffect で同期
|
|
|
+ - handleSubmit でフォーム送信をラップ
|
|
|
+
|
|
|
+5. **誤認識の教訓** ⚠️
|
|
|
+ - PR #10051 の変更ファイルリストを完全に確認せずに「完了」と報告してしまった
|
|
|
+ - **教訓**: 大きな変更の影響範囲は必ず完全にリストアップしてから作業を進める
|
|
|
+ - エンタープライズ向け機能(LDAP, OIDC, SAML)が未移行だったのは重大な見落とし
|
|
|
|
|
|
-## 削除したファイル
|
|
|
+---
|
|
|
|
|
|
-- ❌ `apps/app/src/client/hooks/use-text-input-with-ime.ts` - カスタムフックアプローチを廃止
|
|
|
+## 🚀 次のアクション
|
|
|
|
|
|
-## 修正したファイル
|
|
|
+### 緊急度: HIGH ⚠️
|
|
|
+1. **Phase 5**: SecuritySetting.jsx(1フィールド、最もシンプル)
|
|
|
+2. **Phase 6**: エンタープライズ認証3コンポーネント(36フィールド)
|
|
|
+ - LdapSecuritySettingContents.jsx
|
|
|
+ - OidcSecuritySettingContents.jsx
|
|
|
+ - SamlSecuritySettingContents.jsx
|
|
|
+3. **Phase 7**: LocalSecuritySettingContents.jsx(配列ハンドリング)
|
|
|
|
|
|
-- ✅ `apps/app/src/client/components/Admin/Common/AdminUpdateButtonRow.tsx` - `type` prop を追加(submit/button/reset)
|
|
|
+### 緊急度: LOW
|
|
|
+4. **Phase 8**: Slack 関連3コンポーネント(オプション、すでに useState 使用)
|
|
|
|
|
|
-## 移行対象外(理由付き)
|
|
|
+### その後
|
|
|
+5. **テスト実施** - 移行済みコンポーネントの動作確認(特にエンタープライズ認証)
|
|
|
+6. **ドキュメント化** - 移行ガイドとベストプラクティスの整理
|
|
|
|
|
|
-### 複雑すぎるもの
|
|
|
-- **OidcSecuritySettingContents.jsx** - 10+ の input フィールド、条件付きレンダリング
|
|
|
-- **SamlSecuritySettingContents.jsx** - textarea + 多数の input、複雑なテーブルレイアウト
|
|
|
-- **LdapSecuritySettingContents.jsx** - 10+ の input フィールド、ドロップダウン、条件付きレンダリング
|
|
|
-- **XssForm.jsx** - クラスコンポーネント、ラジオボタン、子コンポーネント、条件付きレンダリング
|
|
|
+---
|
|
|
|
|
|
-### React Hook Form に不適
|
|
|
-- **CustomizeLogoSetting.tsx** - 画像ファイルアップロード、画像切り抜き機能
|
|
|
+## 📌 参考リンク
|
|
|
|
|
|
-### テキスト入力がない
|
|
|
-- **CustomizeFunctionSetting.tsx** - チェックボックスとドロップダウンのみ
|
|
|
-- **CustomizePresentationSetting.tsx** - チェックボックスのみ
|
|
|
+- PR #10051: https://github.com/growilabs/growi/pull/10051/files
|
|
|
+- React Hook Form: https://react-hook-form.com/
|
|
|
+- 関連メモリー: `admin-forms-pr10051-ime-issues.md`
|
|
|
|
|
|
-## ブランチ情報
|
|
|
+---
|
|
|
|
|
|
-- 作業ブランチ: `imprv/admin-form`
|
|
|
-- ベースブランチ: `master`
|
|
|
+## 📈 統計サマリー
|
|
|
|
|
|
-## 参考リンク
|
|
|
+| カテゴリー | 完了 | 未完了 | 合計 | 完了率 |
|
|
|
+|----------|------|--------|------|--------|
|
|
|
+| **PR #10051 ファイル** | 17 | 10 | 27 | 63% |
|
|
|
+| **推定フィールド数** | ~33 | ~48 | ~81 | 41% |
|
|
|
+| **エンタープライズ認証** | 0 | 3 | 3 | **0%** ⚠️ |
|
|
|
+| **その他セキュリティ** | 2 | 2 | 4 | 50% |
|
|
|
+| **Slack 関連** | 1 | 3 | 4 | 25% |
|
|
|
|
|
|
-- React Hook Form 公式: https://react-hook-form.com/
|
|
|
-- Unstated 公式: https://github.com/jamiebuilds/unstated (deprecated)
|
|
|
-- Jotai 公式: https://jotai.org/ (将来的に導入予定)
|
|
|
+**最優先課題**: エンタープライズ認証設定(LDAP, OIDC, SAML)の移行
|