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

卒業生でもホーム画面、プロフィール画面でカレンダーを表示 #3106

Merged
merged 2 commits into from
Aug 10, 2021

Conversation

sota-onozuka
Copy link
Contributor

@sota-onozuka sota-onozuka commented Aug 7, 2021

対象issue

#2983

やったこと

卒業生でもカレンダーを表示すること
プロフィール画面
image

ホーム画面
image

確認してほしい事

  • きちんと表示が変わっているか

確認手順

  1. 卒業生アカウント(id: sotugyou)でログイン
  2. ダッシュボード( http://localhost:3000/ )にてニコニコカレンダーが表示されることを確認
  3. そのまま、もしくは他のアカウントで再度ログインしてsotugyouさんのプロフィール( http://localhost:3000/users/609827778 )に移動
  4. ニコニコカレンダーが表示されることを確認
  • スタイル等崩れていないか
  • コードにおかしなところがないか

よろしくお願いいたします。

@sota-onozuka sota-onozuka requested a review from fugakkbn August 8, 2021 00:37
@sota-onozuka
Copy link
Contributor Author

@FUGA0618 お手数ですが、お手すきの際にレビューをお願いいたします! お忙しいなどございましたら、遠慮なくお教えください!よろしくお願いいたします。

Copy link
Contributor

@fugakkbn fugakkbn left a comment

Choose a reason for hiding this comment

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

実装お疲れさまです!
LGTMです〜👍

1点だけPRのdescriptionについてなのですが、1番頭に

ref: #2983 

のように書いておくとどのissueに対するPRなのかわかりやすいので、そのようにしてもらえたらと思います🙆🏻‍♂️

@sota-onozuka
Copy link
Contributor Author

@FUGA0618 ありがとうございます。頂いたアドバイスを基に、説明を修正させていただきました。次回以降もリンクを張るように意識します!

@sota-onozuka sota-onozuka requested a review from komagata August 8, 2021 23:16
@sota-onozuka
Copy link
Contributor Author

@komagata お手すきの際にレビューをお願いいたします。

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認しました、バッチリです~!

@komagata komagata merged commit 8222e8c into main Aug 10, 2021
@komagata komagata deleted the feature/show-calendar-to-graduates branch August 10, 2021 16:45
@github-actions github-actions bot mentioned this pull request Aug 10, 2021
15 tasks
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.

3 participants