[ 石川さん確認待ち ／ 2人目確認待ち ][ No index ] 検索エンジンがサイトをインデックスしないようにする にチェックが入ってたらアラートを出す機能を追加

[kurudrive]

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

• フルサイトインストーラーでデモサイトは noindex にしたいが、そのままエクスポートするとインポートしたサイトが軒並みnoindex になってしまう。

どういう変更をしたか？

• 検索エンジンがサイトをインデックスしないようにする にチェックが入ってたらアラートを出す機能を追加
  実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

ソースコードについて

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

デザイン・UI

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

プログラムの変更の場合

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

• 書けそうなテストは書いたか？
• 表示要素が仕様通りに表示されない不具合の修正ではない or 表示要素に関する不具合修正の場合テストは書いたか？

その他

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

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

• 検索エンジンがサイトをインデックスしないようにする にチェックを入れて、ダッシュボードのトップにアラートが表示されるかどうか

[goutetsuguma]

以下を確認しました。

• ダッシュボードのホーム（/wp-admin/index.php）でのみアラートが表示されていることが確認できました。
• 設定 > 表示設定 > 検索エンジンがサイトをインデックスしない にチェックを入れるとアラートが表示され、チェックを外すとアラート表示が消えました。
• アラートの閉じる（×）ボタンを押すとアラートが消えることが確認できました。

@kurudrive
動作には問題ありませんが、一度以下のご確認をお願いいたします🙇

[goutetsuguma]

すみません、、AIでコードレビューをしたところ、以下の回答がありました。
回答通りに該当の箇所を削除したところ、閉じるボタンは動くようで問題なさそうでした。
しかしながら、プロジェクト統一の書き方もあると思いますので、さらっと見ていただき検討していただけますと良いかなとおもいました。

WordPressのダッシュボードでは、.notice-dismissボタンの機能は既にWordPressコアによって自動的に処理されています。そのため、以下のjQueryコードは不要です。

echo '<script type="text/javascript">
     (function($){
        $(document).on("click", ".notice-dismiss", function(){
            $(this).closest(".notice").fadeOut();
         });
     })(jQuery);
 </script>';

WordPress の標準の is-dismissible クラスを使う場合は、手動で を追加する必要はなく、WordPress が自動的に閉じるボタンを追加して処理します。以下のコードは不要です。

echo '<button type="button" class="notice-dismiss"><span class="screen-reader-text">' . __( 'Dismiss this notice.', 'vk-all-in-one-expansion-unit' ) . '</span></button>';