bootcamp
bootcamp copied to clipboard
「2回連続sadのユーザー」のカードをVue化する
Issue
- #5415
概要
メンターのダッシュボードに、二回連続sadになった人の日報が表示される場所があります。
この日報のカードをvue化しました。
変更確認方法
- ブランチ
feature/replace-card-user-become-sad-twice-consecutively-with-vue-component
をローカルに取り込む -
rails s
でローカル環境を立ち上げる - kimuraでログインする
- sadの日報を連続になるように2つ作成する(日報が2回連続sadになるように作成する)
- komagataでログインし直す
-
http://localhost:3000/
へアクセス - 2回連続sadの日報がダッシュボードに表示されているか確認する
変更前後での画面変更点
見た目上の変更はありません。
@fuwa-syugyo お疲れ様です! もしお時間がありましたらこちらのレビューをお願いしたいです🙇♀️ 忙しければ別の方にお願いしますので都合が悪ければおっしゃってください〜! よろしくお願いいたします🙇♀️
@AyakaTakashima お疲れ様です!お返事が遅くなり申し訳ございません…! レビューの対応できます!明日を目処に対応します〜
@fuwa-syugyo いえいえ!お忙しいところお引き受けありがとうございます🙇♀️ ゆっくりで大丈夫です☀️よろしくお願いいたします!
@AyakaTakashima
お疲れ様です!
メンターのダッシュボードに二回連続sadになった人の日報が表示されていることを確認しました〜
コンフリクトだけ修正していただければ問題ないと思います!
私からはApproveさせていただきます〜
@fuwa-syugyo ご指摘いただいたコンフリクトを解消いたしました! ご確認ありがとうございます!!!🙇♀️
@komagata お疲れ様です! fuwa-syugyoさんからapproveいただきましたのでレビューをお願いしたいです。 よろしくお願いいたします!
@komagata お疲れ様です! ご指摘いただいた部分、修正致しましたのでご確認お願いいたします!
@AyakaTakashima 「WIP」というコミットメッセージはちょっと意味がわからないのでわかるように別のものにするか、他のコミットと一緒にしてしまうのがいいと思います〜
@komagata WIPは、レビューのために他ブランチに切り替えるために一度コミットしたものなのですが、コミット名を「他ブランチに切り替えるため」と記載してもいいものなのでしょうか。 cafedomancerさんとペアプロした際に「WIPでもなんでもいいので一度コミットしちゃいましょう」というようなことをおっしゃってたことがあったので「WIP」でいいのかと思ってしまいました。
@komagata すみませんが、コミットをまとめようとして変な風になってしまいました。 完了しましたらまたご連絡いたします。
@AyakaTakashima https://github.com/fjordllc/bootcamp/pull/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
をやってみてくださいー。
@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
できませんでした😭
@AyakaTakashima ごめんなさい!ミスです🙇♂️
git checkout main でした🙇♂️
@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
で対応しました!
@komagata 「WIP」というコミットを他のコミットに修正いたしました! ご確認よろしくお願いいたします。
@komagata 修正しました! ご確認よろしくお願いいたします。