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

【確認待ち】GTag対応 #327

Merged
merged 3 commits into from
Mar 26, 2024
Merged

【確認待ち】GTag対応 #327

merged 3 commits into from
Mar 26, 2024

Conversation

mthaichi
Copy link
Contributor

GA4に対応できるようGA設定の部分を変更しました。

  • 「Gtag」を選択できるようにした。
  • 設定値が「G-」「UA-」から始まらない場合は、前に「UA-」をつけたものをIDとする
  • 設定値が「G-」「UA-」から始まる場合は、その文字列をIDとする。

@mthaichi mthaichi requested a review from kurudrive March 23, 2024 13:51
Copy link
Member

@kurudrive kurudrive left a comment

Choose a reason for hiding this comment

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

@mthaichi ありがとうございます。
ここって エスケープしておいた方が良いですよね?
元がしてなかったですけど...

plugins/seo_and_ga/seo_and_ga.php Outdated Show resolved Hide resolved
plugins/seo_and_ga/seo_and_ga.php Outdated Show resolved Hide resolved
@kurudrive kurudrive changed the title 【確認待ち】GTag対応 【丸山さん確認待ち】GTag対応 Mar 23, 2024
@mthaichi mthaichi requested a review from kurudrive March 25, 2024 10:38
@mthaichi
Copy link
Contributor Author

@kurudrive ありがとうございます。おっしゃるとおりでした。
もう一箇所あったので、追加でesc_html 入れました!

@mthaichi mthaichi changed the title 【丸山さん確認待ち】GTag対応 【確認待ち】GTag対応 Mar 25, 2024
@kurudrive
Copy link
Member

@mthaichi ありがとうございまっすー。

スクリーンショット 2024-03-23 10 46 32

@kurudrive kurudrive merged commit ae7e218 into master Mar 26, 2024
@kurudrive kurudrive deleted the feature/ga4 branch March 26, 2024 12:09
@kurudrive
Copy link
Member

@mthaichi もうこの段階で一旦リリースで良いですよね(・w・?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants