-
Notifications
You must be signed in to change notification settings - Fork 2
[ HTMLサイトマップ ] 記事数が0のタクソノミーは表示されないように修正 #1231
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@akito-38
調整ありがとうございます!動作確認問題ありませんでした。
一点だけ、readme.txt の修正お願いします。
readme.txt
Outdated
@@ -71,8 +71,7 @@ e.g. | |||
1. Activate the plugin through the 'Plugins' menu in WordPress | |||
|
|||
== Frequently Asked Questions == | |||
|
|||
|
|||
[ Specification Change ][ sitemap ] Terms and taxonomies with zero articles will no longer be displayed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
「Frequently Asked Questions」に書かれているので「Changelog」の場所に移動お願いします
@akito-38
こちら対応していただけるとよいかと〜 |
@sysbird @goutetsuguma |
ありがとうございます、確認しました。 |
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
#1224
どういう変更をしたか?
タームが登録されてないタクソノミーや、登録されている記事が0のタームしかない場合、
タクソノミー名が表示されないようにしました。
ソースコードについて
デザイン・UI
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
→パス
その他
変更内容について何を確認したか、どういう方法で確認をしたかなど
管理画面で、HTMLサイトマップブロックを配置。
タグが一つも登録されていない状態で、「タグ」と見出しが表示されてないことを確認しました。
タグをいくつか登録して、それらのタグに一つも記事を結び付けてない状態だと「タグ」と見出しが表示されてないことを確認しました。
それらのタグに1つでも記事が結びついてる状態だと「タグ」と見出しが表示され、タグ自体も表示されることを確認しました。
カスタムタクソノミーを作製して、同様に確認しました。
カスタムポストタイプとそれに結びついたカスタムタクソノミーを作製して同様の確認をしました。
レビュワーの確認方法・確認する内容など
実装者と同じ
レビュワーに回す前の確認事項
レビュワー向け
確認して変更が反映されていない場合の確認事項