【2人目確認中】ブログカード 置換する際はURL以外のattributesを削除する #1805
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)
#749 #1778
今後、ブロックバリエーションをユーザーがカスタマイズできる様な機能を考えています
#749 (comment)
その際に元の設定が引き継がれると背景を設定していないバリエーションなのに背景が設定されるなどが発生しユーザービリティーが悪いと思いました
そのためURL以外のattributeはundefinedに設定することにします。
ブロックサポートはstyle以外のattributeも存在していたので確認漏れしてました🙇♂️🙇♂️🙇♂️
現時点では置換ボタンを押したらattributesを削除しないとバリエーションの設定ができないため、バリエーション選択画面で元のブロック設定はクリアされる文言で対応しています。
#749 (comment)
どういう変更をしたか?
url以外のattributesをundefinedにする様に修正
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
ブログカードブロック 追加 #1778 に書いてあるので
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
変更内容について何を確認したか、どういう方法で確認をしたかなど
実装者が確認した手順を箇条書きで記載してください。
確認URL
( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
レビュワーがどういう手順で何を確認して欲しいかを記載してください。
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。