vk-all-in-one-expansion-unit icon indicating copy to clipboard operation
vk-all-in-one-expansion-unit copied to clipboard

【石川さん確認待ち/開発チームレビュー待ち】タクソノミーウィジェットにプルダウンを追加

Open drill-lancer opened this issue 7 months ago • 2 comments

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

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

どういう変更をしたか?

  • タクソノミーウィジェットにプルダウンを追加

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

ソースコードについて

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

デザイン・UI

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

プログラムの変更の場合

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

  • [ ] 書けそうなテストは書いたか? => ドロップダウンを追加したのみなのでスキップ
  • [ ] 表示要素が仕様通りに表示されない不具合の修正ではない or 表示要素に関する不具合修正の場合テストは書いたか?

その他

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

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

  • タクソノミーウィジェットの設定時に リスト / プルダウンを切り替えられる設定が追加されているのを確認
  • タクソノミーウィジェットでリストが選択されているときは今まで通りリスト表示されているのを確認
  • タクソノミーウィジェットでプルダウンが選択されているときはプルダウンが選択されてるのを確認
  • タクソノミーウィジェットのプルダウンでタームを選択すると該当タームのアーカイブページに移動するのを確認

確認URL

ローカル環境

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

  • タクソノミーウィジェットの設定時に リスト / プルダウンを切り替えられる設定が追加されているのを確認
  • タクソノミーウィジェットでリストが選択されているときは今まで通りリスト表示されているのを確認
  • タクソノミーウィジェットでプルダウンが選択されているときはプルダウンが選択されてるのを確認
  • タクソノミーウィジェットのプルダウンでタームを選択すると該当タームのアーカイブページに移動するのを確認

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

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

レビュワー向け

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

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

drill-lancer avatar Jul 11 '24 03:07 drill-lancer