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

「2回連続sadのユーザー」のカードをVue化する #5621

Conversation

AyakaTakashima
Copy link
Contributor

@AyakaTakashima AyakaTakashima commented Oct 3, 2022

Issue

概要

メンターのダッシュボードに、二回連続sadになった人の日報が表示される場所があります。
この日報のカードをvue化しました。
image

image

変更確認方法

  1. ブランチfeature/replace-card-user-become-sad-twice-consecutively-with-vue-componentをローカルに取り込む
  2. rails sでローカル環境を立ち上げる
  3. kimuraでログインする
  4. sadの日報を連続になるように2つ作成する(日報が2回連続sadになるように作成する)
  5. komagataでログインし直す
  6. http://localhost:3000/へアクセス
  7. 2回連続sadの日報がダッシュボードに表示されているか確認する

変更前後での画面変更点

見た目上の変更はありません。

@AyakaTakashima AyakaTakashima force-pushed the feature/replace-card-user-become-sad-twice-consecutively-with-vue-component branch from 4c4fe9a to 5849255 Compare October 21, 2022 11:21
@AyakaTakashima AyakaTakashima marked this pull request as ready for review October 21, 2022 13:25
@AyakaTakashima
Copy link
Contributor Author

@fuwa-syugyo
お疲れ様です!
もしお時間がありましたらこちらのレビューをお願いしたいです🙇‍♀️
忙しければ別の方にお願いしますので都合が悪ければおっしゃってください〜!
よろしくお願いいたします🙇‍♀️

@fuwa-syugyo
Copy link
Contributor

@AyakaTakashima
お疲れ様です!お返事が遅くなり申し訳ございません…!
レビューの対応できます!明日を目処に対応します〜

@AyakaTakashima
Copy link
Contributor Author

@fuwa-syugyo
いえいえ!お忙しいところお引き受けありがとうございます🙇‍♀️
ゆっくりで大丈夫です☀️よろしくお願いいたします!

@fuwa-syugyo
Copy link
Contributor

@AyakaTakashima
お疲れ様です!
メンターのダッシュボードに二回連続sadになった人の日報が表示されていることを確認しました〜
image
コンフリクトだけ修正していただければ問題ないと思います!
私からはApproveさせていただきます〜

@AyakaTakashima AyakaTakashima force-pushed the feature/replace-card-user-become-sad-twice-consecutively-with-vue-component branch from 17c9478 to 85a8246 Compare October 25, 2022 11:01
@AyakaTakashima
Copy link
Contributor Author

@fuwa-syugyo
ご指摘いただいたコンフリクトを解消いたしました!
ご確認ありがとうございます!!!🙇‍♀️

@AyakaTakashima
Copy link
Contributor Author

@komagata
お疲れ様です!
fuwa-syugyoさんからapproveいただきましたのでレビューをお願いしたいです。
よろしくお願いいたします!

@@ -95,6 +97,8 @@ mounter.addComponent(UserMentorMemo)
mounter.addComponent(UserRecentReports)
mounter.addComponent(Talks)
mounter.addComponent(Footprints)
mounter.addComponent(Report)
Copy link
Member

Choose a reason for hiding this comment

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

この行は必要ないかも?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@komagata
ほんとですね🙇‍♀️
当初Report.vueを直接app/views/users/_sad_emotion_report.html.slimに埋め込もうと考えていてそのまま残ってしまってました🙇‍♀️
ご指摘ありがとうございます🙇‍♀️

@AyakaTakashima AyakaTakashima force-pushed the feature/replace-card-user-become-sad-twice-consecutively-with-vue-component branch from 85a8246 to 460639b Compare October 26, 2022 10:18
@AyakaTakashima
Copy link
Contributor Author

@komagata
お疲れ様です!
ご指摘いただいた部分、修正致しましたのでご確認お願いいたします!

@komagata
Copy link
Member

@AyakaTakashima 「WIP」というコミットメッセージはちょっと意味がわからないのでわかるように別のものにするか、他のコミットと一緒にしてしまうのがいいと思います〜

@AyakaTakashima
Copy link
Contributor Author

AyakaTakashima commented Oct 31, 2022

@komagata
WIPは、レビューのために他ブランチに切り替えるために一度コミットしたものなのですが、コミット名を「他ブランチに切り替えるため」と記載してもいいものなのでしょうか。
cafedomancerさんとペアプロした際に「WIPでもなんでもいいので一度コミットしちゃいましょう」というようなことをおっしゃってたことがあったので「WIP」でいいのかと思ってしまいました。

@AyakaTakashima AyakaTakashima force-pushed the feature/replace-card-user-become-sad-twice-consecutively-with-vue-component branch from 460639b to 6905562 Compare October 31, 2022 14:21
@AyakaTakashima
Copy link
Contributor Author

@komagata
すみませんが、コミットをまとめようとして変な風になってしまいました。
完了しましたらまたご連絡いたします。

@machida machida force-pushed the feature/replace-card-user-become-sad-twice-consecutively-with-vue-component branch 2 times, most recently from df0a516 to 8af0e9e Compare November 2, 2022 16:30
@machida
Copy link
Member

machida commented Nov 2, 2022

@AyakaTakashima
#5722 を close して、こちらのブランチを close したものと同じ状態にしました。
こちらで引き続き作業を進めていただけたらと思います。

手元の feature/replace-card-user-become-sad-twice-consecutively-with-vue-component の削除が必要です。

git checkout origin main
git branch -d feature/replace-card-user-become-sad-twice-consecutively-with-vue-component
git fetch origin
git checkout feature/replace-card-user-become-sad-twice-consecutively-with-vue-component

をやってみてくださいー。

@AyakaTakashima
Copy link
Contributor Author

@machida
ご対応ありがとうございます😭
git checkout origin mainを実行すると、下記のようになってしまって実行ができないのですが、git checkout mainとは違うんですよね...?💦

> git checkout origin main
error: pathspec 'main' did not match any file(s) known to git

error: pathspec 'main' did not match any file(s) known to gitについて調べてみると、git fetchすると解決できるという情報を見つけ、やってみましたがgit checkout origin mainできませんでした😭

@machida
Copy link
Member

machida commented Nov 3, 2022

@AyakaTakashima ごめんなさい!ミスです🙇‍♂️

git checkout main でした🙇‍♂️

@AyakaTakashima
Copy link
Contributor Author

@machida
ありがとうございます!
無事下記手順で出来ました🙇‍♀️

> git checkout main
> git branch -D feature/replace-card-user-become-sad-twice-consecutively-with-vue-component
> git fetch
> git checkout feature/replace-card-user-become-sad-twice-consecutively-with-vue-component

git branch -d feature/replace-card-user-become-sad-twice-consecutively-with-vue-componentは、

error: The branch 'feature/replace-card-user-become-sad-twice-consecutively-with-vue-component' is not fully merged.
If you are sure you want to delete it, run 'git branch -D feature/replace-card-user-become-sad-twice-consecutively-with-vue-component'.

というエラーメッセージが出たのでgit branch -D feature/replace-card-user-become-sad-twice-consecutively-with-vue-componentで対応しました!

@AyakaTakashima
Copy link
Contributor Author

@komagata
「WIP」というコミットを他のコミットに修正いたしました!
ご確認よろしくお願いいたします。

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.

conflictの修正をお願いします〜

@AyakaTakashima AyakaTakashima force-pushed the feature/replace-card-user-become-sad-twice-consecutively-with-vue-component branch from 8af0e9e to ce1374a Compare November 7, 2022 14:03
@AyakaTakashima
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.

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

@komagata komagata merged commit 63cfd8e into main Nov 8, 2022
@komagata komagata deleted the feature/replace-card-user-become-sad-twice-consecutively-with-vue-component branch November 8, 2022 05:45
@github-actions github-actions bot mentioned this pull request Nov 8, 2022
16 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.

4 participants