vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

【確認待ち】SNS のシェアボタンの表示ロジックを修正 #1023

Closed drill-lancer closed 1 year ago

drill-lancer commented 1 year ago

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

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

どういう変更をしたか?

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

ソースコードについて

デザイン・UI

プログラムの変更の場合

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

その他

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

  1. ソーシャルボタンの自動挿入を有効化
  2. 適当に投稿タイプ「投稿」で記事を作成
  3. 何もしない状態だと上記投稿で SNS のシェアボタンが表示されるのを確認
  4. シェアボタンを表示しない投稿タイプ「投稿」にチェックを入れて保存
  5. 例の投稿で表示されなくなるのを確認
  6. ②の状態に戻す
  7. 「シェアボタンを表示しない投稿のID」に例の投稿の ID を入力
  8. 例の投稿で表示されなくなるのを確認
  9. 「シェアボタンを表示しない投稿のID」に例の投稿の IDとカンマと投稿が存在する適当な数字を入力
  10. 例の投稿で表示されないままなのを確認
  11. ②の状態に戻す
  12. 例の投稿で「ソーシャルボタンを表示しない」にチェックを入れる
  13. 例の投稿で表示されないのを確認
  14. 表示場所に lightning_comment_before を指定
  15. 3~13 を繰り返して同じ結果になるのを確認

確認URL

ローカル環境

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

  1. ソーシャルボタンの自動挿入を有効化
  2. 適当に投稿タイプ「投稿」で記事を作成
  3. 何もしない状態だと上記投稿で SNS のシェアボタンが表示されるのを確認
  4. シェアボタンを表示しない投稿タイプ「投稿」にチェックを入れて保存
  5. 例の投稿で表示されなくなるのを確認
  6. ②の状態に戻す
  7. 「シェアボタンを表示しない投稿のID」に例の投稿の ID を入力
  8. 例の投稿で表示されなくなるのを確認
  9. 「シェアボタンを表示しない投稿のID」に例の投稿の IDとカンマと投稿が存在する適当な数字を入力
  10. 例の投稿で表示されないままなのを確認
  11. ②の状態に戻す
  12. 例の投稿で「ソーシャルボタンを表示しない」にチェックを入れる
  13. 例の投稿で表示されないのを確認
  14. 表示場所に lightning_comment_before を指定
  15. 3~13 を繰り返して同じ結果になるのを確認

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


レビュワー向け

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

kurudrive commented 1 year ago

@drill-lancer ありがとうございました! 確認手順も明確で良かったです!