bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

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

Open AyakaTakashima opened this issue 2 years ago • 4 comments

Issue

  • #5415

概要

メンターのダッシュボードに、二回連続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 avatar Oct 03 '22 10:10 AyakaTakashima

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

AyakaTakashima avatar Oct 21 '22 13:10 AyakaTakashima

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

fuwa-syugyo avatar Oct 24 '22 12:10 fuwa-syugyo

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

AyakaTakashima avatar Oct 24 '22 12:10 AyakaTakashima

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

fuwa-syugyo avatar Oct 25 '22 05:10 fuwa-syugyo

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

AyakaTakashima avatar Oct 25 '22 11:10 AyakaTakashima

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

AyakaTakashima avatar Oct 25 '22 11:10 AyakaTakashima

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

AyakaTakashima avatar Oct 26 '22 10:10 AyakaTakashima

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

komagata avatar Oct 31 '22 03:10 komagata

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

AyakaTakashima avatar Oct 31 '22 11:10 AyakaTakashima

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

AyakaTakashima avatar Nov 01 '22 09:11 AyakaTakashima

@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 avatar Nov 02 '22 16:11 machida

@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 avatar Nov 03 '22 11:11 AyakaTakashima

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

git checkout main でした🙇‍♂️

machida avatar Nov 03 '22 11:11 machida

@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 avatar Nov 03 '22 11:11 AyakaTakashima

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

AyakaTakashima avatar Nov 03 '22 11:11 AyakaTakashima

@komagata 修正しました! ご確認よろしくお願いいたします。

AyakaTakashima avatar Nov 07 '22 14:11 AyakaTakashima