search

sakura-editor / sakura-editor.github.io

SAKURA Editor portal site
https://sakura-editor.github.io
11 stars 12 forks source link

Google Analytics コードの埋め込みを一元管理できるようにする #38

Closed m-tmatma closed 5 years ago

m-tmatma commented 5 years ago

30: Google Analytics コードの埋め込みを一元管理できるようにする

※ この PR の後に @m-tmatma の Google Analytics コードも足してみようと思います。 ※ HTML ファイル更新用の python はコメントとか手を抜いています。必要なら足します。

berryzplus commented 5 years ago

※ この PR の後に @m-tmatma の Google Analytics コードも足してみようと思います。

一緒にいれなくて動作確認大丈夫ですか？

変更提案の内容はOKで、 @m-tmatma さんのアカウント組み込むのもOKです。 懸念点は「これ入れた動作確認って @kobake さんしかできないんじゃ？」ってとこです。

m-tmatma commented 5 years ago

懸念点は「これ入れた動作確認って @kobake さんしかできないんじゃ？」ってとこです。

そうなんですよね。

変更提案の内容はOKで、 @m-tmatma さんのアカウント組み込むのもOKです。

これを入れ取るときも誰か別の方の ID も同時に追加したいですね。

一緒にいれなくて動作確認大丈夫ですか？

あとこの対応 feature/google-analytics-external-js のブランチに対して適用しているのですが master にマージする前にテストする方法ってあるんですかね。

一応同じロジックで (sakura-editor.github.io ではなく) 自分の GitHub Pages のサイトで動作するのは確認しています。 (リアルタイムの画面で、テスト用のアクセスをするページがあって、そこからアクセス させたときに、現在のユーザー数が 1 になって、テスト用のクライアントからみたいなのが 出たので正しく動作してると思います)

m-tmatma commented 5 years ago

マージしちゃいます

m-tmatma commented 5 years ago

なんかあったらまたPR出す感じでお願いします。いちおう「コードだけ読むだけチェック」をして通してく感じにしますので。

なにかありましたので、 #39 を送りました。