vektor-inc / vk-blocks-pro

13 stars 3 forks source link

[スライダー ( slider )] 0 系統の deprecated を削除 #2319

Closed MasayaMORIMOTO closed 1 week ago

MasayaMORIMOTO commented 2 weeks ago

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

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

どういう変更をしたか?

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

実装者の確認事項

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

プログラムの変更の場合

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

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

  1. 編集画面で「スライダー」を追加し、「スライダーアイテム」を追加。
  2. 「スライダー」のエディタ設定、高さ、スライド設定、アイテムの複数表示設定をして保存。
  3. 「スライダーアイテム」のレイアウト設定、背景設定をして保存。
  4. フロントエンドで表示確認。

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

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

  1. 編集画面で「スライダー」を追加し、「スライダーアイテム」を追加してください。
  2. 「スライダー」のエディタ設定、高さ、スライド設定、アイテムの複数表示設定をして保存します。
  3. 「スライダーアイテム」のレイアウト設定、背景設定をして保存します。
  4. フロントエンドで表示確認してください。

レビュワー向け

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

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

drill-lancer commented 1 week ago

問題ないようでしたので承認しました。


どなたか2人目確認お願いします。

akito-38 commented 1 week ago

@MasayaMORIMOTO @drill-lancer すみません。 save の deprecated の数と hook の deprecated の数が異なると思うのですが、 これは問題ありませんでしょうか?

image

mthaichi commented 1 week ago

@MasayaMORIMOTO @drill-lancer すみません。 save の deprecated の数と hook の deprecated の数が異なると思うのですが、 これは問題ありませんでしょうか?

@akito-38 横から失礼しますー。 ここの配列の数が deprecatedの個数と合致していればよいので、問題ないと思いますよ。

https://github.com/vektor-inc/vk-blocks-pro/blob/f21b311558e4a6fdfa8bb55412acb74d781f81a9/src/blocks/slider/deprecated/hooks/index.js#L10-L21

akito-38 commented 1 week ago

@MasayaMORIMOTO @drill-lancer @mthaichi

そうなんですね。 ありがとうございます。 では、他の部分も問題無いと思いますので、マージしておきます。