-
Notifications
You must be signed in to change notification settings - Fork 71
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
お試し延長一覧ページのタイトルをh1に変更し他のタブと統一 #7463
Conversation
レビュー依頼お世話になります!本日からチーム開発に参加しました🙏 お忙しい中、申し訳ありませんがレビューをお願いすることは可能でしょうか? 内容はgood first issueなのでシンプルなもので、大きく時間はかからない内容かと思います🙏 ご検討のほど、よろしくお願いいたします🙇♂️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
お疲れ様です!
コードはバッチリOKです👍
その他、細かいですがPRについて2点気になったのでお伝えさせていただきます。
1点目は、Issue番号です。
#7367
が記載されていますが、おそらく#7369
ではないでしょうか。
こちらは混乱防止のため修正をお願いいたします🙏
2点目は、Screenshotについてです。
- 変更前後で同じ構図で撮る
- 加工して変更点を強調する
とするとより良くなるのかなと思いました。
↓ 加工はこんな感じです
リリースノートを書く際にもスクショが必要になるので、よろしければやってみてください!
繰り返しになりますが、コードはOKですので、私からはApproveとさせていただきます😄
これから一緒にがんばっていきましょう~💪✨
ご丁寧にありがとうございます🙏
こちらタイポしてました!見つけていただき助かりました👀早速こちら修正しています!
こちらもたしかに構図が同じかつ、マークがあると確認する側も見やすいですね🧐アドバイスを反映させて、次に進めたいと思います! お忙しい中、丁寧にみていただきありがとうございました✨引き続き一緒に頑張っていきましょう💪 |
レビュー依頼お疲れ様です! チームメンバーによるレビューが終了しました! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
確認させて頂きました。OKです〜🙆♂️
Issue
概要
お試し延長一覧ページのタイトル(「管理者ページ」)をh2タグからh1タグに変更するissue。
変更確認方法
feature/trial-extension-list-update-title
をローカルに取り込むforeman start -f Procfile.dev
でサーバーを起動Screenshot
変更前
変更後