shinya

継続して検討。メリット・デメリットがそれぞれあるので、一概に決めきれない部分あり。

実装の検討は引き続き進めたいが、決めないといけない仕様の部分が多岐にわたるので、4.2.1のスコープに含めるかどうかはもう少し議論が必要。

@watarukashii お手数ですが、上記レビュー内容の反映をお願いできればと思います。

レビューの指摘事項、こちらで修正いたしました。 （テストも一部追加しました）

@chihiro-adachi ご指摘ありがとうございます。承知です。 まだ @kiy0taka さんの指摘があるのでapprove頂き次第消すようにします。

現在、TwitterとFacebookの追加を頂いておりますが、この2つで良いかという点と Twitter→Xに変更しているという点から 追加の改修がまだ必要だと感じています。 また別の問題として、そもそも需要がないというのもあります 参考： https://twitter.com/EC_CUBE/status/1765283570686521420 こちらもう少しディスカッションを重ねたいです。

コードの最新化後、動作確認が必要。 @refine-kenshi テストコードの追加をまたお時間有るときにお願いしたいです。 難しそうならこちらで巻き取ります。

今回、4.3へのアップデートなので取り込み対象にします。 ただし、ブロックを新しく追加する形に変更したいと思います。 そのためソースコードをいくつか変更することになるかと思います。 また、性能に変化が出そうな部分になるのでその辺りも考慮に入れて改修を行います。

こちら新しい要素として書き換えました。 （デフォルトのブロックとしては入れずに手動で追加すれば表示される） 確認よろしくお願いします。

ソースコードを現最新Versionに更新した後、Conflictを解消しマージするように動きます。