【2人目確認中】広告アラートの投稿タイプのチェックボックスを修正 - 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

【2人目確認中】広告アラートの投稿タイプのチェックボックスを修正 #1044

Closed drill-lancer closed 1 year ago

drill-lancer commented 1 year ago

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

フォーラム

どういう変更をしたか？

広告アラートの投稿タイプのチェックボックスに不備があったのを修正しました。

ソースコードについて

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[x] 関数名 / 変数名 / クラス名 / 保存値名はそれだけで内容が想像できるものになっているか？紛らわしい命名になっていないか？
[x] 関数名 / 変数名 / クラス名 / 保存値名は既存のコードの命名規則に沿ったものになっているか？

デザイン・UI

[x] 初見のユーザーが予備知識無しで使っても使いやすいようになっているか？
[x] 情報意味を考慮した意味グルーピング・余白になっているか？
[x] アラートの表示など追加した場合は他の同様の表示と同じデザインになっているか？

プログラムの変更の場合

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

[ ] 書けそうなテストは書いたか？ ⇒チェックボックスの内容なので省略

その他

[x] readme.txt に変更内容は書いたか？
[x] Files changed (変更ファイル)の内容は目視でちゃんと確認したか？
[x] このチェック項目を機械的にチェックするのではなく本当にちゃんと確認をしたか？
[x] レビュワーが確認しないでリリースしてしまっても問題ないレベルまでちゃんと作りこみ・確認をしたか？

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

投稿タイプ「投稿」で広告アラートを表示するように設定を変更
投稿タイプ「投稿」で新規投稿を作成した後広告アラートの設定を「共通」のまま公開
広告アラートが表示されるのを確認
投稿タイプのチェックボックスの部分のソースコードを見て不備がないことを確認

確認URL

ローカル環境

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

投稿タイプ「投稿」で広告アラートを表示するように設定を変更
投稿タイプ「投稿」で新規投稿を作成した後広告アラートの設定を「共通」のまま公開
広告アラートが表示されるのを確認
投稿タイプのチェックボックスの部分のソースコードを見て不備がないことを確認

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

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

レビュワー向け

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

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

goutetsuguma commented 1 year ago

アラートが出ることを確認できました！

@drill-lancer 別件で質問なのですが、カスタム投稿タイプには個別の記事からの表示・非表示の設定は表示されない　であっていますか？

kurudrive commented 1 year ago

カスタム投稿タイプには個別の記事からの表示・非表示の設定は表示されない　であっていますか？

いや、あってないですね...。カスタム投稿にも投稿個別で表示・非表示設定は表示されていないとだめかと...。

まぁそれは他の問題なので別 issue で。