[カード ( card )] 0 系統の deprecated を削除 - Githubissues

vektor-inc / vk-blocks-pro

13 stars 3 forks source link

[カード ( card )] 0 系統の deprecated を削除 #2320

Closed MasayaMORIMOTO closed 2 weeks ago

MasayaMORIMOTO commented 2 weeks ago

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

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

どういう変更をしたか？

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

実装者の確認事項

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

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[x] Files changed (変更ファイル)の内容は目視で確認したか？
[ ] readme.txt に変更内容は書いたか？ → スキップ
[x] 本当にちゃんと確認をしたか？

プログラムの変更の場合

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

[ ] 書けそうなテストは書いたか？ → スキップ

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

編集画面で「カード」を追加。
「カードアイテム」の URL、リンクターゲット、REL 属性を設定して保存。
フロントエンドで表示確認。

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

[x] 実装者はこのテンプレートのチェック項目をちゃんと確認してチェックしたか？

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

編集画面で「カード」を追加してください。
「カードアイテム」の URL、リンクターゲット、REL 属性を設定して保存します。
フロントエンドで表示確認してください。

レビュワー向け

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

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

プルしたか？
ビルドしたか？
ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
npm install したか？
composer install したか？
キャッシュをクリアして確認したか？

drill-lancer commented 2 weeks ago

save の deprecated の数と hook の deprecated の数は後々合わせる必要があるのでこちらで合わせて置きました。それ以外は問題なかったので承認します。

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

MasayaMORIMOTO commented 2 weeks ago

save の deprecated の数と hook の deprecated の数は後々合わせる必要があるのでこちらで合わせて置きました。 @drill-lancer RICK さん、フォローありがとうございました！