ブロックを各機能ごとに分離

drill-lancer commented 1 year ago

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

該当する機能が有効化されていないのにその機能のブロックが設置できてしまうのを修正

どういう変更をしたか？

ブロックは該当する機能が読み込まれていないと配置できないようにしました。
ブロックの JS は公式の翻訳で行えば OK のような気がするので一旦 language フォルダを削除しました。

レビューに回す前に確認する事

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[x] Files changed (変更ファイル)の内容は目視で確認したか？
[ ] readme.txt に変更内容は書いたか？ ⇒ ブロックの有効・無効を本来の形にしただけなのであえて書くべき内容でもないような・・・
[x] 本当にちゃんと確認をしたか？

プログラムの変更の場合

[ ] 書けそうなテストは書いたか？ ⇒下記確認事項がエラーなく実行できれば OK なので不要かと ( 既存の PHP Unit で十分かと )

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

１つ１つ機能を有効化・無効化して機能が有効なときのみブロックが使えることを確認しました。

[x] SNS
[x] child-page-index
[x] contact-section
[x] page-list-ancestor
[x] sitemap-page
[x] call-to-action

確認URL

ローカル環境

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

１つ１つ機能を有効化・無効化して機能が有効なときのみブロックが使えることを確認お願いします。

[x] SNS
[x] child-page-index
[x] contact-section
[x] page-list-ancestor
[x] sitemap-page
[x] call-to-action

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

[ ] このテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー向け

確認して変更が反映されていない場合の確認事項

プルしたか？
ビルドしたか？
ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
npm install したか？
composer install したか？

kurudrive commented 1 year ago

@drill-lancer develop マージしたらテストコケるようになっちゃったのでお手数ですが調整よろしくお願いいたしますー！ ※本当は僕のしごとのような気もするけど書籍の都合で僕のスケジュールが破綻したから助けて (´；ω；｀)

drill-lancer commented 1 year ago

@kurudrive テストを通して置きました。

kurudrive commented 1 year ago

@drill-lancer ありが...Playwriteのテストが落ちるようになってますー。

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/actions/runs/4070041261/jobs/7010530847

CTA未登録時にCTAブロックを配置すると「No CTA registered.」が表示されないといけないのに「Please Select ... 」が表示されるようですー。

drill-lancer commented 1 year ago

@kurudrive どうやら石川さんが develop ブランチにマージしたときに CTA の index.php がハチャメチャなことになっていてそれについては現在修正中です。

drill-lancer commented 1 year ago

@kurudrive 簡易的にですが直しておきました。

drill-lancer commented 1 year ago

@kurudrive そのとおりですね。見落としていましたので修正しました。

kurudrive commented 1 year ago

@drill-lancer ありがとうございました！

vektor-inc / vk-all-in-one-expansion-unit