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

管理者で入った相談部屋の一覧にページャーを追加 #4118

Merged
merged 7 commits into from
Feb 8, 2022

Conversation

ot0m1
Copy link
Contributor

@ot0m1 ot0m1 commented Feb 2, 2022

ref: #4066

要件

管理者でログインし、左のナビの相談に入ると、相談部屋一覧が表示される。相談部屋は数が多いのでページャーを付ける。一覧に表示させる数は日報や提出物と同じでOK。

画面イメージ

スクリーンショット 2022-02-05 12 38 28

スクリーンショット 2022-02-05 12 39 03

確認方法

  1. feature/add-pager-to-list-of-consultation-room ブランチをローカルに持ってくる(参考:https://qiita.com/great084/items/ad74dd064a2c2bc47cff
git fetch origin pull/{このプルリクエストのid}/head:{任意のブランチ名}
git checkout {上記でfetchした任意のブランチ名}
  1. 管理者(komagata or machida)のアカウントでログイン
  2. 相談部屋一覧( http://localhost:3000/talks )にアクセスしてがページャーが表示、動作することを確認

@ot0m1
Copy link
Contributor Author

ot0m1 commented Feb 5, 2022

@Aseiide
お疲れ様ですレビューお願いいたします。

@ot0m1 ot0m1 requested a review from Aseiide February 5, 2022 03:57
Copy link
Contributor

@Aseiide Aseiide left a comment

Choose a reason for hiding this comment

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

vueとjson.jbuilderについては僕自身あまり分かってないのですが、動作確認は問題なくできましたし、コードも問題ないように思えますのでLGTMです!
Draftになってるので、実装が終わってチームメンバー/komagataさんにレビューを送るときはDraftから外しちゃったほうが良いと思います!
lgtm

@ot0m1 ot0m1 marked this pull request as ready for review February 5, 2022 12:48
@ot0m1
Copy link
Contributor Author

ot0m1 commented Feb 5, 2022

@Aseiide レビューありがとうございました!Draftご指摘ありがとうございます
@komagata レビューお願いいたします

@ot0m1 ot0m1 requested a review from komagata February 5, 2022 12:52
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 e53446a into main Feb 8, 2022
@komagata komagata deleted the feature/add-pager-to-list-of-consultation-room branch February 8, 2022 12:19
@github-actions github-actions bot mentioned this pull request Feb 8, 2022
54 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