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

研修生にもニコニコカレンダーを表示する #7492

Merged
merged 1 commit into from
Mar 9, 2024

Conversation

a-terumoto-gs
Copy link
Contributor

@a-terumoto-gs a-terumoto-gs commented Mar 5, 2024

Issue

概要

ユーザーの詳細画面で現役生にはニコニコカレンダーが表示されますが、
研修生の場合は表示されていませんでした。
そのため、研究生にもニコニコカレンダーが表示されるようにしました

変更確認方法

  1. feature/display-niconico-calendar-to-traineesをローカルに取り込む
  2. foreman start -f Procfile.devでサーバーを立ち上げる
  3. http://localhost:3000/users?target=trainee にアクセスする
  4. 研修生の中から任意のユーザーをクリックする
  5. ニコニコカレンダーが表示されるのを確認する

Screenshot

変更前

右側には何も表示されない
image

変更後

現役生と同様にニコニコカレンダーが表示されるように
image

@a-terumoto-gs a-terumoto-gs requested a review from hirano-vm4 March 5, 2024 06:02
@a-terumoto-gs a-terumoto-gs marked this pull request as ready for review March 5, 2024 06:02
@a-terumoto-gs
Copy link
Contributor Author

@hirano-vm4
お疲れ様です!
こちらのissueのレビューをお願いしてもよろしいでしょうか?
全く急ぎではないのでお手すきの際にお願いできればと思います。
よろしくお願いいたしますm(__)m

@hirano-vm4
Copy link
Contributor

@a-terumoto-gs

お世話になります!先日からチーム開発に参加しました🙏

レビュー依頼ありがとうございます!
数日以内に確認して、お返しします🙆よろしくお願いします〜

Copy link
Contributor

@hirano-vm4 hirano-vm4 left a comment

Choose a reason for hiding this comment

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

@a-terumoto-gs

お待たせいたしました!
バッチリ表示されていて良い感じだと思いましたので、Approveさせてもらいました🙆

@a-terumoto-gs a-terumoto-gs requested a review from komagata March 8, 2024 06:28
@a-terumoto-gs
Copy link
Contributor Author

@hirano-vm4
レビューありがとうございました。
迅速な対応に感謝します(*'▽')

@komagata
チームメンバーの方にApproveいただいたのでレビューをお願いいたしますm(__)m

@a-terumoto-gs a-terumoto-gs self-assigned this Mar 8, 2024
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.

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit a5dc9a4 into main Mar 9, 2024
9 checks passed
@komagata komagata deleted the feature/display-niconico-calendar-to-trainees branch March 9, 2024 17:35
@github-actions github-actions bot mentioned this pull request Mar 9, 2024
29 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