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.vue,watches.vueをVueMounterに対応させた #5503

Merged
merged 5 commits into from
Sep 14, 2022

Conversation

keiz1213
Copy link
Contributor

@keiz1213 keiz1213 commented Sep 8, 2022

issue

概要

watch.vue,watches.vueのマウント方法を従来の方法からVueMounterで一元的にマウント管理する方法に変更しました。
見た目上の変更はありません。

変更確認方法

  1. chore/watches-for-vue_mounterをローカルに取り込む
  2. rails sで立ち上げる
  3. kimuraでログインする
  4. http://localhost:3000/current_user/watches にアクセスする
  5. watch一覧が表示されていることを確認する(kimuraは32件watchしています)

@keiz1213 keiz1213 marked this pull request as ready for review September 11, 2022 02:28
@keiz1213 keiz1213 self-assigned this Sep 11, 2022
@keiz1213
Copy link
Contributor Author

@choco0809
お疲れさまです!@choco0809さんが最近VueMounterに取り組まれていたのでレビュー依頼させていだきました🙏
全然急いでないのでお時間があります時にレビューいただけますでしょうか?よろしくお願いいたします。

@choco0809
Copy link
Contributor

@keiz1213
お疲れ様です🙇‍♂️🙇‍♂️🙇‍♂️
本日中に確認いたしますので、少々お待ちください~
取り急ぎの返信まで。

Copy link
Contributor

@choco0809 choco0809 left a comment

Choose a reason for hiding this comment

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

@keiz1213
遅くなってしまい申し訳ございません🙇‍♂️
コード、動作ともに問題はなかったのですがconflictが発生しているように思えますので、
そちらだけ解消していただければと思います🙇‍♂️🙇‍♂️🙇‍♂️

@keiz1213
Copy link
Contributor Author

@choco0809
コンフリクト修正とCIが通りました😄確認よろしくお願いいたします!🙏

@choco0809 choco0809 self-requested a review September 13, 2022 13:07
Copy link
Contributor

@choco0809 choco0809 left a comment

Choose a reason for hiding this comment

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

@keiz1213
コード、動作ともに問題ないことを確認しましたので、Approveさせていただきます!!

@keiz1213
Copy link
Contributor Author

@choco0809
確認ありがとうございました!🙏

@komagata
こちらメンバーの方からapproveしていただきましたので確認お願いします!

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 9a82d93 into main Sep 14, 2022
@komagata komagata deleted the chore/watches-for-vue_mounter branch September 14, 2022 07:00
@github-actions github-actions bot mentioned this pull request Sep 14, 2022
20 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