vektor-inc / vk-blocks-pro

13 stars 3 forks source link

[Outer ( outer )] 0 系統の deprecated を削除 #2329

Closed MasayaMORIMOTO closed 2 days ago

MasayaMORIMOTO commented 1 week ago

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

https://github.com/vektor-inc/vk-blocks-pro/issues/2297

どういう変更をしたか?

・deprecated にある ver0 系統のフォルダを削除しました。 ・テスト用のファイルを削除しました。

実装者の確認事項

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

プログラムの変更の場合

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

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

  1. 編集画面で「Outer」を追加。
  2. 背景設定、レイアウト設定、区切りの設定、枠線の設定、コンテナ内側のスペース設定、最小高さ設定をして保存。
  3. フロントエンドで表示確認。

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

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

  1. 編集画面で「Outer」を追加します。
  2. 背景設定、レイアウト設定、区切りの設定、枠線の設定、コンテナ内側のスペース設定、最小高さ設定をして保存します。
  3. フロントエンドで表示確認してください。

レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

akito-38 commented 1 week ago

@MasayaMORIMOTO こちらも https://github.com/vektor-inc/vk-blocks-pro/pull/2328#issuecomment-2485921825 と同様に以下変数を纏められそうですが、どうなんでしょう? 「blockAttributes」「blockAttributes2」「blockAttributes3」

MasayaMORIMOTO commented 4 days ago

@akito-38 src/blocks/_pro/outer/deprecated/save/index.js の 「blockAttributes」「blockAttributes2」「blockAttributes3」をまとめました。 (数字を詰めて、全体的に修正しています) ご確認をお願いいたします。