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

Watchタブでも日報作成とマイプロフィールの確認をしたい #5758

Merged
merged 1 commit into from
Nov 29, 2022

Conversation

ksmxxxxxx
Copy link
Contributor

@ksmxxxxxx ksmxxxxxx commented Nov 10, 2022

Issue

概要

ダッシュボードのトップで表示している「マイプロフィール」と「日報作成」のリンクを、Watch中タブでも表示する

変更点確認方法

  1. feature/add_create_daily_note_button_on_watch_viewブランチをローカルに取り込む
  2. bin/rails sでローカルサーバーを起動
  3. kimuraでログイン
  4. ダッシュボード→Watchタブをクリック。「マイプロフィール」、「日報作成」ボタンが追加されていることを確認。
  5. kimuraからログアウト→ advijirouでログイン
  6. ダッシュボード→Watchタブをクリック。「マイプロフィール」のみ表示されることを確認。

変更前

image

変更後

kimura アカウント
image

advijirou アカウント
image

@ksmxxxxxx
Copy link
Contributor Author

@shirotamaki お疲れさまです。
Pull Requestを作成しましたので、レビューしていただけますでしょうか?
よろしくおねがいします〜 🙏🏻

@shirotamaki
Copy link
Contributor

@ksmxxxxxx お疲れ様です!
レビューご依頼いただきありがとうございます!日曜までお時間いただけますと幸いです。よろしくお願いします〜🙏

Copy link
Contributor

@shirotamaki shirotamaki left a comment

Choose a reason for hiding this comment

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

@ksmxxxxxx
お疲れさまです!
動作確認できました!LGTMです👍 Approveいたします〜🙆‍♂️

@ksmxxxxxx
Copy link
Contributor Author

@shirotamaki レビューありがとうございます〜
@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.

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

@komagata komagata merged commit fb08119 into main Nov 29, 2022
@komagata komagata deleted the feature/add_create_daily_note_button_on_watch_view branch November 29, 2022 21:21
@github-actions github-actions bot mentioned this pull request Nov 29, 2022
18 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.

[ダッシュボード]Watch中にリンクを追加
3 participants