vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

CTA ブロックを追加 #844

Closed drill-lancer closed 2 years ago

drill-lancer commented 2 years ago

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

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/issues/843

どういう変更をしたか?

レビューに回す前に確認する事

プログラムの変更の場合

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

  1. npm run build
  2. 予め CTA をいくつか用意(ブロックエディタパターンおよび従来のパターン)
  3. 上記の両方のパターンが正常に表示されるかを確認

確認URL

なし(ローカル環境)

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

  1. npm run build
  2. 予め CTA をいくつか用意(ブロックエディタパターンおよび従来のパターン)
  3. 上記の両方のパターンが正常に表示されるかを確認

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


レビュワー向け

確認して変更が反映されていない場合の確認事項

kurudrive commented 2 years ago

@drill-lancer 僕の環境だと本文欄は問題ないですが、サイトエディタではどうもブロックエラーになりまする。 明日以降で良いのでよろしくお願いいたしますー。

TypeError: Cannot read properties of undefined (reading 'vkexunit_cta_each_option')
    at edit (block.min.js?ver=9.78.1.0:6:3312)
    at ct (react-dom.min.js?ver=17.0.1:9:43430)
    at os (react-dom.min.js?ver=17.0.1:9:111136)
    at Ur (react-dom.min.js?ver=17.0.1:9:77643)
    at Ir (react-dom.min.js?ver=17.0.1:9:77571)
    at Dr (react-dom.min.js?ver=17.0.1:9:77434)
    at Pr (react-dom.min.js?ver=17.0.1:9:74429)
    at react-dom.min.js?ver=17.0.1:9:30173
    at unstable_runWithPriority (react.min.js?ver=17.0.1:9:7431)
    at xn (react-dom.min.js?ver=17.0.1:9:29950)
drill-lancer commented 2 years ago

@kurudrive エラー修正しました!

kurudrive commented 2 years ago

@drill-lancer マージしましたー!ありがとうございます!