商品詳細ページにSNS共有ボタンを表示する

[matsuoshi]

概要(Overview・Refs Issue)

商品詳細ページほかに、SNSシェアボタンを追加します

#4849

方針(Policy)

Twitter および Facebook のシェアボタンを、各ページに配置します。
ただし、以下のページでは表示されません。

• マイページ系
• カートページ系
• 会員登録ページ系

実装に関する補足(Appendix)

シェアボタンは新規ブロック social_buttons.twig として作成し、デフォルト状態ではレイアウトのフッター上部に入るように設定しています。

CSS はスタイルガイド内の "2.3 その他ボタン" 内に追加しました。

テスト（Test)

期待するページに、SNSシェアボタンのブロックが出力されていることを確認する Webテストを追加しました。

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[chihiro-adachi]

@matsuoshi
このあたりを参考に、マイグレーションの作成もお願いできますか？
https://github.com/EC-CUBE/ec-cube/blob/4.0/app/DoctrineMigrations/Version20201218044542.php

[matsuoshi]

@chihiro-adachi 確認ありがとうございます、マイグレーションファイルを追加しました

[kiy0taka]

マイグレーションでエラーになりました。

An exception occurred while executing 'INSERT INTO dtb_block (
            id, device_type_id, block_name, file_name, create_date, update_date, use_controller, deletable, discriminator_type
        ) VALUES(
            ?, 10, "ソーシャルボタン", "social_buttons", "2021-02-12 14:00:00", "2021-02-12 14:00:00", 0, 0, "block"
        )' with params [15]:
SQLSTATE[42703]: Undefined column: 7 ERROR:  column "ソーシャルボタン" does not exist
LINE 4:             $1, 10, "ソーシャルボタン", "social_buttons", "2...
                            ^

[matsuoshi]

@kiy0taka すいません、マイグレーションのエラーを修正しました

[k-yamamura]

デフォルト状態から表示されるようになっていますが利用しないサイトもありますので、
デフォルトは未設定が良いと思います。

The base branch was changed.

[shinya]

現在、TwitterとFacebookの追加を頂いておりますが、この2つで良いかという点と
Twitter→Xに変更しているという点から
追加の改修がまだ必要だと感じています。

また別の問題として、そもそも需要がないというのもあります
参考： https://twitter.com/EC_CUBE/status/1765283570686521420

こちらもう少しディスカッションを重ねたいです。