[ カスタム投稿タイプマネージャー ] 設定から解説記事やベクトレへのリンクを追加 #1117
Conversation
mtdkei
changed the title
mtdkei
requested review from
mthaichi,
kurudrive,
sysbird,
goutetsuguma,
akito-38,
drill-lancer,
kaorock72 and
MasayaMORIMOTO
and removed request for
kaorock72
|
@mtdkei |
|
@drill-lancer |
|
@mtdkei |
drill-lancer
changed the title
|￣￣￣￣────￣￣￣￣）工数かけてレビューしてるのでせっかく気づいたんですから普通に「これ不自然じゃない？」って言ってください！気づくのは素晴らしい事デス。 |
|
@drill-lancer @kurudrive |
mtdkei
changed the title
…-in-one-expansion-unit-dismiss'
mtdkei
changed the title
|
ExUnitのところを途中からVK Blocks Proと勘違いしておりましてお手数をおかけいたしました。 |
kurudrive
changed the title
| <?php echo esc_html( $att['name'] ); ?> | ||
| </a></span> | ||
| <a href="<?php echo ( $att['url'] ) ? esc_url( $att['url'] ) : admin_url() . 'admin.php?page=vkExUnit_main_setting'; ?>" | ||
| <?php echo ( isset( $att['target'] ) && $att['target'] === '_blank' ) ? 'target="_blank"' : ''; ?>> |
There was a problem hiding this comment.
@mtdkei
あー、これは「ExUnit の有効化設定画面で別ウィンドウで開く設定を追加」で趣旨が少し異なるので、微妙なラインですが別のプルリクであげて欲しいですー。で、多分こういう変更は作業中に発生してすぐに承認して欲しい時が多いと思うので、別でプルリクだけ作って、説明とかは適当で良いので、優先ですぐ見て欲しいとか連絡いただけるといいかなとー。
（今回はこのままでいいですー🙂）
There was a problem hiding this comment.
そうですね、おっしゃる通りです。次から別のプルリクで対応します。
| // サイトの言語が日本語 (ja) であるかどうかを判定 | ||
| if ( get_locale() !== 'ja' ) { | ||
| return ''; // 日本語以外の場合は何も返さない | ||
| } | ||
|
|
||
| // ユーザーが通知を無視したフラグが保存されているかをチェック | ||
| if ( get_user_meta( get_current_user_id(), 'vk-all-in-one-expansion-unit_dismissed_notice', true ) ) { | ||
| return ''; // 通知を無視している場合は何も返さない | ||
| } |
There was a problem hiding this comment.
@mtdkei おや？
// サイトの言語が日本語 (ja) であるかどうかを判定
if ( get_locale() !== 'ja' ) {
return ''; // 日本語以外の場合は何も返さない
}
// ユーザーが通知を無視したフラグが保存されているかをチェック
if ( get_user_meta( get_current_user_id(), 'vk-all-in-one-expansion-unit_dismissed_notice', true ) ) {
return ''; // 通知を無視している場合は何も返さない
}
の部分の判定処理は is_display_help_notice() の中に移動でいいんじゃないかなと
思いましたがどうでしょう（・ｗ・？
There was a problem hiding this comment.
@kurudrive
すみません、そのチェックを取り込むのが抜けていました。is_に取り込みました。日本語サイトのみ表示され、通知を無視をクリックすると表示されなくなることを確認しました。
There was a problem hiding this comment.
There was a problem hiding this comment.
すみません、プッシュミスでした。先ほドプッシュしました。
|
@mtdkei ありがとうございます。 本来なら VK_Post_Type_Manager::is_display_help_notice() のテストを書くところですが...まぁ本質的な機能の部分ではないからスキップでいいかな... という事でマージしますん。 |
kurudrive
changed the title
チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）
#1099
どういう変更をしたか？
'target' => '_blank',がない場合は、テキストリンククリック後、リンクURLが同一タブで開きます。)実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
ソースコードについて
デザイン・UI
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
書いていない場合は書かない理由を記載してください。
その他
変更内容について何を確認したか、どういう方法で確認をしたかなど
WordPressの言語設定が日本語の場合、以下を確認しました。
確認URL
（ どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL ）
レビュワーの確認方法・確認する内容など
レビュワーに回す前の確認事項
レビュワー向け
確認して変更が反映されていない場合の確認事項