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

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

Open
wants to merge 6 commits into
base: 4.3
Choose a base branch
from

Conversation

matsuoshi
Copy link
Contributor

概要(Overview・Refs Issue)

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

#4849

方針(Policy)

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

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

実装に関する補足(Appendix)

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

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

テスト(Test)

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

相談(Discussion)

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

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

レビュワー確認項目

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

@matsuoshi matsuoshi added the enhancement 機能追加 label Feb 15, 2021
@okazy okazy added affected:template フロントテンプレートの変更 css CSSの修正が必要なPR/Issue。スタイルガイドへの反映が必要 labels Feb 15, 2021
@okazy okazy added this to the 4.1 milestone Feb 15, 2021
@@ -13,3 +13,4 @@ id,device_type_id,block_name,file_name,create_date,update_date,use_controller,de
12,10,News,news,2017-03-07 10:14:52,2017-03-07 10:14:52,0,0,block
13,10,Product Search,search_product,2017-03-07 10:14:52,2017-03-07 10:14:52,1,0,block
14,10,Featured,topic,2017-03-07 10:14:52,2017-03-07 10:14:52,0,0,block
15,10,Social Buttons,social_buttons,2021-02-12 14:00:00,2021-02-12 14:00:00,0,0,block
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

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...
                            ^

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

kiy0taka
kiy0taka previously approved these changes Mar 16, 2021
@chihiro-adachi chihiro-adachi modified the milestones: 4.1, 4.1.x Sep 3, 2021
@chihiro-adachi chihiro-adachi changed the base branch from 4.1-feature to 4.1 September 6, 2021 05:20
@k-yamamura
Copy link
Contributor

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

@xuelian311 xuelian311 modified the milestones: 4.1.x, 4.2.2 May 8, 2023
@xuelian311 xuelian311 modified the milestones: 4.2.2, 4.2.x May 23, 2023
@xuelian311 xuelian311 modified the milestones: 4.2.x, 4.2.3 Jul 7, 2023
@xuelian311 xuelian311 modified the milestones: 4.2.3, 4.2.x Aug 25, 2023
@xuelian311 xuelian311 modified the milestones: 4.2.x, 4.3.0 Feb 15, 2024
@shinya shinya changed the base branch from 4.1 to 4.3 March 6, 2024 07:43
@shinya shinya dismissed kiy0taka’s stale review March 6, 2024 07:43

The base branch was changed.

@xuelian311 xuelian311 removed this from the 4.3.0 milestone Apr 11, 2024
@xuelian311 xuelian311 added this to the 4.3.x milestone Apr 11, 2024
@shinya
Copy link
Contributor

shinya commented Apr 17, 2024

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:template フロントテンプレートの変更 css CSSの修正が必要なPR/Issue。スタイルガイドへの反映が必要 enhancement 機能追加 Status: discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants