# ジャンプさんMTG ## 本日確認したいこと :::info デザイン: <font color="tomato">すみません、ご共有いただけると幸いです。</font> 公開日: <font color="tomato">3/1までで変更なしでしょうか?</font> 進行スケジュールの予定: <font color="tomato">←田邊さんから伺ってもよいでしょうか?</font> コードレビューのタイミング: <font color="tomato">本日相談</font> PR発行の方法とお願い事項: ::: ### 本日のMTGあと弊社からご共有すること - 作業ブランチ名: 支給したブランチ上で作業お願いします! - PR発行時の件名 - GA計測用のid名: ※コンテンツ内容によって要否がかわります。 ----- ## コードレビューについて ### コードレビューのステップ Step1: ローカル環境チェック(コードレビュー0.5) Step2: 開発環境チェック(コンテンツ面の確認) Step3: コードレビュー(プルリクエスト発行) ↑Step2の戻し状況等をみつつ、合格の目処がついたら、こちらのほうで発行を依頼します。 ### コードレビューでみたいこと。 ![](https://i.imgur.com/3uf7wrZ.png) ※ ③④は静的サイトなので、負荷が少ないという前提 **** [Step1: ローカル環境チェック]で①の全部と②の半分をみる、 [Step3: コードレビュー(プルリクエスト発行)]②③④をみるイメージ。 ## ご相談 Step1: ローカル環境チェック(コードレビュー0.5) <font color="tomato">←2/10にPR発行いただく</font> Step2: 開発環境チェック(コンテンツ面の確認) <font color="tomato">←2/18 開発環境アップ〜オハヨー社内確認</font> Step3: コードレビュー(プルリクエスト発行) <font color="tomato">↑2/10に発行いただく</font> 番外編: プルリクエストの発行方法 [ここから](https://ap-northeast-1.console.aws.amazon.com/codesuite/codecommit/repositories/ohayo-milk.co.jp/pull-requests?region=ap-northeast-1&status=OPEN&pull-requests-meta=eyJmIjp7InRleHQiOiIifSwicyI6e30sIm4iOjEwLCJpIjowfQ) **件名** 依頼や変更があった際に気付けるよう、通知イベントを設定しています。 こちらからお伝えする件名で発行をお願いします:bow: ## お伝えできていなかったコーディングルール… README.mdをご参照ください:bow: ### アクセス解析のための実装留意点 GAでアクセス解析をする、イベントトラッキングの情報を把握しやすくするために、ブロックごとにidを付与しています。 ※ 上記の理由から装飾目的でのid利用を禁止しています。装飾で使用する場合はclass指定でお願いします。 なお、今回計測が必要となった場合は、別途イベントトラッキング計測のための仕様を共有させていただきます。