[ CTA ] Fix No CTA Random Save / CTAが存在しない状態でランダム指定で保存しようとするとエラーになるので修正 - Githubissues

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

wordpress plugin of powerful support

https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/

GNU General Public License v2.0

7 stars 1 forks source link

[ CTA ] Fix No CTA Random Save / CTAが存在しない状態でランダム指定で保存しようとするとエラーになるので修正 #905

Closed kurudrive closed 1 year ago

kurudrive commented 1 year ago

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

CTAが存在しない状態でランダム指定で保存しようとするとエラーになる不具合を修正

どういう変更をしたか？

CTAが存在しない場合に投稿を保存しようとした時に発生するエラー対応その旨メッセージとCTA登録ページへのリンクを表示するように変更
表示指定してあったCTAがなくなってる場合はその旨メッセージを表示

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

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

プログラムの変更の場合

[x] 書けそうなテストは書いたか？

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

未登録処理確認

CTAが未登録の状態で、CTAブロックを配置 → 登録ないぞメッセージが表示される

登録済みが削除されたケース

適当にCTAを作作成
どこかの投稿にCTAブロックを配置して先程作成したCTAを指定
元のCTAを削除
投稿に配置してあったCTAブロックを確認 → 指定されたCTAが存在しないよメッセージが表示されている事を確認

npm install
npm run build
wp-env start
npm run phpunit
npx playwright test --trace on --project=chromium

↑ 新規WordPressの新規投稿の時にブロックエディタへようこそのダイアログが出る場合と出ない場合があって状況によってテストがコケるので、 /tests/e2e/eta.spec.ts の 27行目前後をコメントアウト・解除してください https://github.com/vektor-inc/vk-all-in-one-expansion-unit/blob/5be5678ec6cc6e3960fa8274ddb33be20118be57/tests/e2e/cta.spec.ts#L27

確認URL

ローカルで確認よろしくお願いいたします。

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

Playwrite で e2e テストを書いてるのでなんとなくみてみてください。
現状アラートが js 側で処理しているものと、php側で処理している部分とわかれてしまっている。もし改善できそうな点があれば...

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

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

レビュワー向け

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

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

drill-lancer commented 1 year ago

CTA を無効化してもブロックの PHP は読み込まれず JS は読み込まれるという状態になります。 JS をバラバラにしてそれぞれの機能が有効な状態のみブロックを読み込むという処置が必要と思われます。