akito-38
commented
2 weeks ago すみません。 lint通したら全部変更したようになってますが、実際の変更内容は96行目を追加しただけです。
Open akito-38 opened 2 weeks ago
akito-38
commented
2 weeks ago すみません。 lint通したら全部変更したようになってますが、実際の変更内容は96行目を追加しただけです。
MasayaMORIMOTO
commented
3 days ago @akito-38
動作には関係ないのですが、下記の記述が他の記述と異なっているのが気になりました。
（コーディング規約の問題です）
修正をお願いできればと思います。
src/blocks/_pro/post-category-badge/index.php の Line: 96 です。
if(!$taxonomy->hierarchical) continue;
mthaichi
commented
3 days ago ありがとうございます。確認いたしました。 2人目の確認をお願いいたします。 @mthaichi #2313 (comment) の見解から、カテゴリーバッジブロックにタグが出ているのが仕様と異なる気がしたのですが、こちらは合っているか念のためご確認いただけたらと思います。
@mtdkei 「カテゴリーバッジ / タグ」ブロックは出ないのが仕様ですので、今回の修正は適切だと思います。
mthaichi
commented
3 days ago @akito-38 動作には関係ないのですが、下記の記述が他の記述と異なっているのが気になりました。 （コーディング規約の問題です） 修正をお願いできればと思います。 src/blocks/_pro/post-category-badge/index.php の Line: 96 です。
if(!$taxonomy->hierarchical) continue;
@akito-38 WordPressのコーディング規約に則ってください。則ってないと composer phpcs でエラーが出て、どこが違うのか確認できます。
https://ja.wordpress.org/team/handbook/coding-standards/wordpress-coding-standards/php
・・・あれ？プッシュする時に composer phpcsが走ってエラーになるはずですが、でなかったです？
akito-38
commented
2 days ago @mthaichi
ご指摘ありがとうございます。 こちら対応しておきました。
composer phpcsを実行してみると改行コードについてのエラーも出ていて、 それを修正したらまた全てが変更箇所になってしましました。
実際に変えたのは95～97行目です。 確認お願いいたします。
あとプッシュする時のエラーについては、覚えてないですが見落としたのかもしれません。すみません。 別のissues対応等でpushするとき、改めてよく確認してみます。
チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）
https://github.com/vektor-inc/vk-blocks-pro/issues/2313
どういう変更をしたか？
バリエーションで、カテゴリーバッジ / タグ ブロックが作られていたので、作られないように変更
スクリーンショットまたは動画
変更前 Before
変更後 After
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
変更内容について何を確認したか、どういう方法で確認をしたかなど
カテゴリーバッジ / タグ ブロックが表示されてないことを確認。 カテゴリーバッジ及び、カテゴリーバッジ / カテゴリーブロッグを配置して問題無く表示されるか確認。
*
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
実装者と同じ
*
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。