search

sakura-editor / sakura-editor.github.io

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

Google Analytics の config 部の所有者名 (kobake) をコメントで追記 #19

Closed kobake closed 6 years ago

kobake commented 6 years ago

Google Analytics の config の UA-120820034-1 の所有者が kobake であることを後から見て分かりやすいようにコメント入れました。

KENCHjp commented 6 years ago

コメントだけの問題ですが、このつけ方だと、「kobakeが修正しました」みたいに読めるので、 「kobakeのidです」というのがわかったほうがせっかくのコメントの為にはいいかも。

kobake commented 6 years ago

たしかに。修正します～

kobake commented 6 years ago

コメントだけの問題ですが、このつけ方だと、「kobakeが修正しました」みたいに読めるので、 「kobakeのidです」というのがわかったほうがせっかくのコメントの為にはいいかも。

これの対応入れました！

KENCHjp commented 6 years ago

＞ gtag('config', 'UA-120820034-1'); // kobake 所有の Google Analytics tracking code

これ以上ないほどわかりやすいですvv ああでも一か所にまとめたい。

kobake commented 6 years ago

一か所にまとめたい。

たしかに。

一旦この PR は所有者の明記対応だけにしておいて、別件 PR で定義場所の集約を試してみる感じにしたいです。

kobake commented 6 years ago

レビューありがとうございます！自己マージ完。