-
Notifications
You must be signed in to change notification settings - Fork 71
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
reports/reportのvueをvue mounterに対応させた #5344
Conversation
5950509
to
ca74dd2
Compare
@komagata ローカルのメインブランチを汚染してしまい、修正しようとするとコンフリクトするので諦めて、プルリクをチェリーピックで作り直しました。チームメンバーレビューは以前のブランチでapporove受けています。 レビューをお願いしてもよろしいでしょうか? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
conflictの修正をお願いします〜
ca74dd2
to
7146ad5
Compare
@komagata これでどうでしょうか?? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
確認させて頂きました。OKです〜🙆♂️
Issue
概要
report.vue, reports.vueをVueMounterに対応させる
すでにマージ済みのプルリク#5113 を参考にしました
実装確認方法
chore/reports-transition-for-vue_mounter
ブランチを取り込む。一般ユーザーでログイン
以下のルートで日報のカードコンポーネントが従来通り表示出来ていることを確認する(カッコ内は権限指定。権限未記載は権限に指定無し)
3.1 reports
3.2 users/:user_id/reports
3.3 companies/:companies_id/reports
3.4 current_user/reports
3.5 reports/:id の直近の日報(一般ユーザー)
3.6 reports/:id の直近の日報(メンター)
3.7 reports/unchecked(メンター)
3.8 talks/:user_id のユーザーの日報タブ(管理者)
その他
practice/:id/reports、メンターダッシュボードの2回連続悲しい顏ユーザーはvue未対応なので、今回の修正の対象外
(他のreportsがVueMounter対応になった後も、上記2つが正常に表示出来ていることを確認している)