vektor-inc / vk-blocks-pro

13 stars 3 forks source link

[タイムライン( timeline )] 0 系統の deprecated を削除 #2333

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. 編集画面で「タイムライン」を追加し、「タイムラインアイテム」をいくつか追加。
  2. 「タイムライン要素」のラベル、色、スタイルを設定。
  3. フロントエンドで表示確認。

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

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

  1. 編集画面で「タイムライン」を追加し、「タイムラインアイテム」をいくつか追加します。
  2. 「タイムライン要素」のラベル、色、スタイルを設定してください。
  3. フロントエンドで表示確認します。

レビュワー向け

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

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

akito-38 commented 1 week ago

@MasayaMORIMOTO こちらも https://github.com/vektor-inc/vk-blocks-pro/pull/2328#issuecomment-2485921825 と同様に変数を纏められそうなので、お手数ですが対応が必要か確認をお願いいたします。

MasayaMORIMOTO commented 4 days ago

@akito-38 src/blocks/_pro/timeline-item/deprecated/index.js の 変数をまとめました。 ご確認をお願いいたします。