【確認済】get_balloon_meta_optionsデフォルト値の取得方法調整 #1380
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
#1363 の吹き出し版
どういう変更をしたか?
option値のデフォルト値を設定する箇所が2箇所に別れていたため、見通しが悪いかつミスを起こしやすいので調整します
デフォルト値を取得するメソッドは #1300 このブランチで作成しました。
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
事前に
test-vk-blocks-options.php
に書いてあります変更内容について何を確認したか、どういう方法で確認をしたかなど
以下、レビュー確認方法同様です。
確認URL
( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
・テストコマンドで確認
・管理画面でdevelopブランチと同様の動作,保存がされるか確認
・他、気になる箇所や影響がありそうなところなどあったらご指摘お願いします。
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。