【確認待ち】広告アラートを追加

[drill-lancer]

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

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/issues/1005

どういう変更をしたか？

• 広告アラートを追加

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

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

プログラムの変更の場合

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

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

1. ExUnit の機能設定で Promotion Alert を有効化
2. Promotion Alert の項目を適当に設定
3. 適当に投稿記事を作成し Promotion Alert の表示・非表示を設定して公開
4. 仕様通り・想定通りに動作しているのを確認
5. npm run phpunit

確認URL

ローカル環境

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

1. ExUnit の機能設定で Promotion Alert を有効化
2. Promotion Alert の項目を適当に設定
3. 適当に投稿記事を作成し Promotion Alert の表示・非表示を設定して公開
4. 仕様通り・想定通りに動作しているのを確認
5. npm run phpunit

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

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

---

レビュワー向け

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

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

[goutetsuguma]

@drill-lancer

少し触ってみた感想です。

• Atert Text / Atert Content / Atert Hook とありますが、Alertの間違いですかね、、？

• Atert Content　の入力欄下に補足文章がいりそうです。 例：* 「アラートコンテンツ」に入力がある場合は「アラートテキスト」は表示されず、「アラートコンテンツ」に入力した内容に上書きされます。

• Atert Hook　の入力欄下にに補足文章がいりそうです。 例：* 通常本文の最初に自動的に挿入されます。挿入箇所をアクションフックで指定したい場合はアクションフック名を入力してください。

• Atert Content　の中に入れた文章は、改行したい場合は<br>タグを自分で書いて挿入する感じでしょうか？改行したら自動で改行されると嬉しいかもしれないけど難しそうでしょうか。難しいようでしたらこのままで大丈夫です！

• 個別記事のアラートの表示/非表示の出し分けはできるのでしょうか？

【追記】

• .veu-promotion-alertというクラス名がつきますが、他のをみると.veu_でveu の後にづつく記号がアンダーバーだったりするのですが、promotion-alertのはハイフンになっているようです。そのあたりのルールとかはあるのでしょうか？なければ大丈夫です🙇‍♀️ちょっと気になりましたので書いてみました

[kurudrive]

.veu-promotion-alertというクラス名がつきますが、他のをみると.veu_でveu の後にづつく記号がアンダーバーだったりするのですが、promotion-alertのはハイフンになっているようです。そのあたりのルールとかはあるのでしょうか？なければ大丈夫です🙇‍♀️ちょっと気になりましたので書いてみました

現状は veu_ でよろしくお願いいたします。

[drill-lancer]

@goutetsuguma @kurudrive

Atert Text / Atert Content / Atert Hook とありますが、Alertの間違いですかね、、？

修正しました

Atert Content　の入力欄下に補足文章がいりそうです。 例：* 「アラートコンテンツ」に入力がある場合は「アラートテキスト」は表示されず、「アラートコンテンツ」に入力した内容に上書きされます。

追加しました

Atert Hook　の入力欄下にに補足文章がいりそうです。 例：* 通常本文の最初に自動的に挿入されます。挿入箇所をアクションフックで指定したい場合はアクションフック名を入力してください。

追加しました

Atert Content　の中に入れた文章は、改行したい場合は
タグを自分で書いて挿入する感じでしょうか？改行したら自動で改行されると嬉しいかもしれないけど難しそうでしょうか。難しいようでしたらこのままで大丈夫です！

普通に HTML タグが使えます。@kurudrive さんによればのちのちこれ用のパターンを作るとか 作らないとか・・・ 説明も追加しました。

個別記事のアラートの表示/非表示の出し分けはできるのでしょうか？

設定画面は作成済みです。こちらの設定が優先されます。 説明も一応追加しました。

.veu-promotion-alertというクラス名がつきますが、他のをみると.veu_でveu の後にづつく記号がアンダーバーだったりするのですが、promotion-alertのはハイフンになっているようです。そのあたりのルールとかはあるのでしょうか？なければ大丈夫です🙇‍♀️ちょっと気になりましたので書いてみました

修正しました。

[goutetsuguma]

@drill-lancer @kurudrive リックさん修正ありがとうございます！説明が追加されてめっちゃわかりやすくなりました！！htmlタグが入るので使いやすいと思います！ アクションフックも、個別ページの表示非表示も問題なさそうです。

あとは、フックの補足文が、上部に表示されていましたのでAlert Hook の下に移動お願いします。（添付画像参照）

---

【すこし質問です】 ExUnitの「メイン設定」>「Promotion Alert」の「Display Post Types」の、例えば「post」にチェックをつけた場合は、新規で作成した記事のみ広告アラートメッセージが表示される仕様であっていますか。（過去の記事にはつかないので個別の記事設定から表示にしてもらう仕様） もしそうであれば、なにか注釈とか必要そうですかね？？ 注釈例：*ここにチェックをいれた投稿タイプは、新規で投稿する記事にのみ広告アラートが表示されます。過去の記事は個々の記事の「プロモーションアラート設定」から設定してください。

なぜかというと、postにチェック入れたら全記事アラート文が入るのかと思う人もいるのかなと思いまして。（私自身がそうおもったため汗）

[drill-lancer]

あとは、フックの補足文が、上部に表示されていましたのでAlert Hook の下に移動お願いします。

修正しました。

ExUnitの「メイン設定」>「Promotion Alert」の「Display Post Types」の、例えば「post」にチェックをつけた場合は、新規で作成した記事のみ広告アラートメッセージが表示される仕様であっていますか。

過去記事も対象になるように調整しました。

[goutetsuguma]

@drill-lancer ありがとうございます！！全記事に表示されているのを確認しました！

あと何度もすみません、、！ 今気づいたのですが、「Call To Action」の「表示場所（オプション）」のところのデザインと合わせた方が良い気がしましたので、以下のように調整することは可能でしょうか？こちらの方がオプション感がでるので、分かりやすそうなのと、デザインの共通のため。

私の作業範囲の場所がありましたら教えてください🙇‍♀️

参考：「Call To Action」の「表示場所（オプション）」

---

【修正前】

【修正後】

[drill-lancer]

@goutetsuguma @kurudrive 修正しました。

[goutetsuguma]

@drill-lancer @kurudrive

修正ありがとうございます！確認しました。

お手隙の時に石川さん、確認をお願いいたします。

[kurudrive]

@drill-lancer @goutetsuguma

ありがとうございます。

1. ExUnit肥大化問題があり、使わない機能のCSSを読み込むのは良くないので、基本的には今後 vkExUnit_style.scss からの読み込みは停止していきたいなと思っております。 で、今回CSSも少ないので PHPのインラインCSSで読み込んじゃってもいいんじゃないかなと。

2. 個別の設定は ExUnit 共通metabox内に入れたいです。

[drill-lancer]

@kurudrive 調整しました。