Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

編集画面のエラーの数を減少 #220

Merged
merged 3 commits into from
Jan 27, 2025
Merged

Conversation

drill-lancer
Copy link
Member

@drill-lancer drill-lancer commented Oct 25, 2024

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

#219

どういう変更をしたか?

コンソールエラーの表示数を減らしました

実装者の確認事項

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

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

プログラムの変更の場合

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

  • 書けそうなテストは書いたか?
    ⇒編集画面のコンソールエラーを修正したのみなので省略

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

コンソールエラーの数が減ったのを確認しました。

実装前
before

実装後
after

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

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

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

コンソールエラーの数が減ったのを確認お願いします。


レビュワー向け

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

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

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

@kurudrive
Copy link
Member

@drill-lancer ありがとうございます。
減るには減っていますが...残りは...無理そう的な感じなのでしょうか?
一旦ここまででマージして別ブランチで対応する感じでしょうか?

@drill-lancer
Copy link
Member Author

@kurudrive
VK Block Patterns を無効化しても出るものがあるのでこれ以上は無理そうです。

@kurudrive
Copy link
Member

kurudrive commented Oct 25, 2024

いや、無効化しても出てるものは仕方ないのですが、
2つ目のやつは VK Block Patterns が原因で出てて、K Block Patterns を無効化すると消えるような気が...(・w・;

@drill-lancer
Copy link
Member Author

@kurudrive
VK Block Patterns 側で createRoot を react-dom/client から読み込もうとしても
WordPress 側で react-dom から読み込まれていてそれを再利用している空気を感じます。
よってこれが限界です。

@kurudrive
Copy link
Member

これが限界です。

@mthaichi @MasayaMORIMOTO @akito-38

同意見でしょうか?

@MasayaMORIMOTO
Copy link

MasayaMORIMOTO commented Nov 2, 2024

@drill-lancer @kurudrive
本件、調査してみました。
以下の点から考えて、RICK さんの「これが限界」説に同意見です。
(考慮漏れのご指摘があれば、是非お願いします!)

(1) コードの変更箇所の記述に問題がない点
(2) 同様の変更を行うべきファイルが他に見当たらない点
(3) WP のバージョンを 6.5.5 にダウングレードすると、Warning が表示されなくなる点
↓ 6.5.5 の場合の編集画面のコンソール出力です
スクリーンショット 2024-11-02 14 13 22

@goutetsuguma goutetsuguma changed the title 【確認待ち】編集画面のエラーの数を減少 【意見待ち?/ 確認待ち】編集画面のエラーの数を減少 Dec 4, 2024
@kurudrive kurudrive changed the title 【意見待ち?/ 確認待ち】編集画面のエラーの数を減少 編集画面のエラーの数を減少 Jan 27, 2025
@kurudrive kurudrive merged commit 6362d2e into master Jan 27, 2025
9 checks passed
@kurudrive kurudrive deleted the fix/reacr-worning branch January 27, 2025 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

編集画面でエラーが出ているので修正したい
3 participants