【2人目確認待ち】[ 投稿リスト / 投稿リストスライダー ] 表示条件をコンポーネント化

[mtdkei]

チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）

#2334

どういう変更をしたか？

投稿リスト、投稿リストスライダーの編集画面にある「表示条件」設定をコンポーネント化しました。
1人は開発の方に確認いただきたく思います。

スクリーンショットまたは動画

内部的な変更のためスクショ等はありません。

変更前 Before

変更後 After

実装者の確認事項

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

• 複数の意図の変更 （ 機能の不具合修正 + 別の機能追加など ） を含んでいないか？
• Files changed (変更ファイル)の内容は目視で確認したか？
• readme.txt に変更内容は書いたか？
• 本当にちゃんと確認をしたか？

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。

• 書けそうなテストは書いたか？
  → 内部の変更のためスキップ。必要そうでしたらおっしゃってください。

変更内容について何を確認したか、どういう方法で確認をしたかなど

投稿リスト、投稿リストスライダーの新規、既存ブロックにおいて以下を確認しました。

• 表示条件 パネルでさまざまな設定を施し、編集画面、フロントエンドともに正常に設定できることを確認しました。https://www.vektor-inc.co.jp/service/wordpress-plugins/vk-blocks/vk-blocks-blocks/post-list/
  (「投稿タイプを全て外した時の挙動」と「常に最初の投稿から表示する」の不具合は本プルリク以外にも発生するため別でissue化します。)
• 表示条件以外のパネル(表示タイプとカラム、表示要素)で設定を変えた時、編集画面、フロントエンドともに正常に表示できることを確認しました。
• 分割読み込み、Tree Shaking(Lightinigのみ)、Preload(Lightinigのみ)のON、OFFともに確認しました。

レビュワーに回す前の確認事項

• 実装者はこのテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー確認方法・確認内容など

実装者と同じ確認を行ってください。
他にも気になることがありましたら見ていただけたら幸いです。

---

レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

• プルしたか？
• ビルドしたか？
• ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
• npm install したか？
• composer install したか？
• キャッシュをクリアして確認したか？

[mthaichi]

@mtdkei 修正ありがとうございます！

let postTypesProps = vk_block_post_type_params.post_type_option;

こういう外部から値を受け取る処理は共通コンポーネントの中にいれないでください。
Edit側に上記の記述を残し、Propsで受け取ってください。

コンポーネントの入力値がまったく同じなら、結果は同じであることが望ましいですね。

なお、EditでPropsで参照されている vkBlockParam はどこで使ってますか？ 結局使われてない感じがします。上記のことをやろうとした名残でしょうか。

ぱっと見ただけでちゃんと見れてないのですが、取り急ぎ上記ご確認ください。

[mtdkei]

@mthaichi
ありがとうございます。ご指摘の内容を確認し、以下の通り進めてみました。

• 外部依存の処理（vk_block_post_type_params）は共通コンポーネントに含めず、Edit側で処理した結果をDisplayConditionコンポーネントにPropsとして渡すように修正しました。
• postTypesPropsとtermsByTaxonomyNameを利用し、全体的にコードを整理しました。
• 以前の名残と思われるvkBlockParamや使われていない値は削除しました。

[mthaichi]

@mtdkei ありがとうございます！
問題ないと思いますので、Approveします。

どなたか2人目チェックお願いします。

[mthaichi]

@mtdkei あ、コンフリクトだけ解消頂いて、「2人目確認待ち」にしてください。🙏

[mtdkei]

@mthaichi
すみません、ありがとうございます。ただいま解消しました！
コンクリフトしてた投稿リストスライダーの編集画面部分の動きは軽くみたのですが、もし何かありましたらご連絡ください。