vk-all-in-one-expansion-unit
vk-all-in-one-expansion-unit copied to clipboard
[ 石川さん確認待ち / 2人目確認待ち ][ No index ] 検索エンジンがサイトをインデックスしないようにする にチェックが入ってたらアラートを出す機能を追加
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
- フルサイトインストーラーでデモサイトは noindex にしたいが、そのままエクスポートするとインポートしたサイトが軒並みnoindex になってしまう。
どういう変更をしたか?
- 検索エンジンがサイトをインデックスしないようにする にチェックが入ってたらアラートを出す機能を追加 実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
ソースコードについて
- [ ] 複数の意図の変更 ( 機能の不具合修正 + 別の機能追加など ) を含んでいないか?
- [ ] 関数名 / 変数名 / クラス名 / 保存値名 はそれだけで内容が想像できるものになっているか?紛らわしい命名になっていないか?
- [x] 関数名 / 変数名 / クラス名 / 保存値名 は既存のコードの命名規則に沿ったものになっているか?
デザイン・UI
- [x] 初見のユーザーが予備知識無しで使っても使いやすいようになっているか?
- [ ] 情報意味を考慮した意味グルーピング・余白になっているか?
- [ ] アラートの表示など追加した場合は他の同様の表示と同じデザインになっているか?
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。 書いていない場合は書かない理由を記載してください。
- [ ] 書けそうなテストは書いたか?
- [ ] 表示要素が仕様通りに表示されない不具合の修正ではない or 表示要素に関する不具合修正の場合テストは書いたか?
その他
- [x] readme.txt に変更内容は書いたか?
- [x] Files changed (変更ファイル)の内容は目視でちゃんと確認したか?
- [x] このチェック項目を機械的にチェックするのではなく本当にちゃんと確認をしたか?
- [x] レビュワーが確認しないでリリースしてしまっても問題ないレベルまでちゃんと作りこみ・確認をしたか?
変更内容について何を確認したか、どういう方法で確認をしたかなど
- 検索エンジンがサイトをインデックスしないようにする にチェックを入れて、ダッシュボードのトップにアラートが表示されるかどうか