vektor-inc / vk-blocks-pro

11 stars 3 forks source link

【2人め確認待ち】[ リンクツールバー ] 編集画面でのCORSエラーを回避するように修正しました #2250

Open mtdkei opened 5 days ago

mtdkei commented 5 days ago

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

2246

どういう変更をしたか？

外部リンク設定時にCORSエラーが発生する問題を修正。外部リンクに対してはプレビューやメタデータの取得（タイトル・ファビコン取得）を行わず、CORSエラーを回避するように変更しました。

スクリーンショットまたは動画

変更前 Before

スクリーンショット 2024-10-01 10 15 02

変更後 After

スクリーンショット 2024-10-01 10 13 24

変更後にも発生している、/wp-json/wp/v2/block-patterns/patterns?_locale=user:1 の500エラーは今回のとは直接関係がないと考えられるためそのままにしてます。もし関係ありそうでしたら教えていただけたら幸いです。

実装者の確認事項

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

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

プログラムの変更の場合

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

[ ] 書けそうなテストは書いたか？ → スキップしてますが、もし必要でしたらご連絡ください。

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

外部リンク設定時にCORSエラーが発生しないことを確認。
内部リンク設定時にタイトルやファビコンが正しく表示されることを確認。
URL入力欄に保存後、フロントエンドでリンク設定が反映されることを確認。

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

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

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

実装者と同じ確認を行ってください。何か気づいたことがあれば適宜追加確認をお願いいたします。

レビュワー向け

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

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

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

sysbird commented 5 days ago

さっそく対応いただいてありがとうございます解決されているのを確認しました！