Conversation
## Walkthrough
Vue.jsのシングルファイルコンポーネント「following.vue」、Reactコンポーネント「MentorCourses.jsx」および関連する読み込み用プレースホルダーコンポーネントが削除されました。また、APIコントローラー「API::CoursesController」とその統合テストも削除され、メンターコース一覧はサーバーサイドレンダリングのHTMLテーブルに置き換えられました。
## Changes
| ファイル/パス | 変更内容概要 |
|------------------------------------------------|----------------------------------------------------------------------|
| app/javascript/following.vue | ユーザーのフォロー・ウォッチ管理用Vueコンポーネントを削除 |
| app/javascript/components/MentorCourses.jsx | Reactによるメンターコース一覧コンポーネントを削除 |
| app/javascript/components/LoadingMentorPageCoursesPlaceholder.jsx | メンターコース読み込み用プレースホルダーReactコンポーネントを削除 |
| app/javascript/components/MentorPageLoadingView.jsx | メンターコース読み込み用テーブル表示Reactコンポーネントを削除 |
| app/controllers/api/courses_controller.rb | API::CoursesControllerを削除 |
| test/integration/api/courses_test.rb | API::CoursesControllerの統合テストを削除 |
| app/views/mentor/courses/index.html.slim | ReactコンポーネントからサーバーサイドレンダリングのHTMLテーブルへ置換 |
## Suggested reviewers
- e-yanagita-gs
- komagata
## Poem
> さよならVueの小さな友、
> フォローとウォッチを見守った日々。
> ボタンのラベルも、APIの道も、
> ふわふわ消えて、春風に舞う。
> 新しい朝に、ぴょんと跳ねて、
> コードの野原でまた会おう！
> 🐇✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@thmz337 |
|
こんにちは。レビューさせていただきますね |
|
@thmz337 |
|
お疲れ様です。確認しているのですが、イシューを見ると実装の置き換えも要求されているような気がします。 ファイルの削除のみで良いのでしょうか？ |
|
ご確認いただきありがとうございます！ しかし調査を進めたところ、フォロー機能は既に以下のファイルを中心とした、Railsのview + Vanilla JS形式での実装の置き換えが完了していました。
さらに詳しく調べた結果、今回削除対象の そのため、今回のプルリクエストでは、実装の置き換えは行わず、不要となっているファイルの削除のみを行った、という経緯になります。 ご確認のほど、よろしくお願いいたします。 |
|
ご丁寧にありがとうございます🙇 またご連絡させていただきますね。 |
#7426に対応箇所を見つけることができました。 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
app/controllers/api/courses_controller.rb(0 hunks)app/javascript/components/LoadingMentorPageCoursesPlaceholder.jsx(0 hunks)app/javascript/components/MentorCourses.jsx(0 hunks)app/javascript/components/MentorPageLoadingView.jsx(0 hunks)app/views/mentor/courses/index.html.slim(1 hunks)test/integration/api/courses_test.rb(0 hunks)
💤 Files with no reviewable changes (5)
- app/controllers/api/courses_controller.rb
- app/javascript/components/LoadingMentorPageCoursesPlaceholder.jsx
- test/integration/api/courses_test.rb
- app/javascript/components/MentorPageLoadingView.jsx
- app/javascript/components/MentorCourses.jsx
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build_and_test
🔇 Additional comments (2)
app/views/mentor/courses/index.html.slim (2)
29-52: PRの目的と実際の変更内容に不整合がありますPRの目的は「ユーザーのフォロー表示に関わる不要なvueファイルの削除」となっていますが、実際の変更内容はメンターコース一覧のReactコンポーネントをサーバーサイドレンダリングのHTMLテーブルに置き換える内容になっています。この不整合について確認が必要です。
Likely an incorrect or invalid review comment.
29-39: HTMLテーブル構造の実装は適切ですセマンティックなHTMLテーブル構造で実装されており、ヘッダー行とCSS クラスの使用も適切です。管理画面のテーブルとして必要な要素が含まれています。
|
すいません！間違ってこちらのブランチにPRを投げてしまいました...。 |
|
revertさせていただきました。こちら別の対応が良い場合はご教示いただけますでしょうか。 |
|
@thmz337 revertは緊急的にリリースを元に戻すときなど稀な時にしか使わないやつです。 |
|
@e-yanagita-gs さんの方ではできることはないかなと思います。 @thmz337 こちら @e-yanagita-gs さんの作業に影響が出ているかもなので優先して対応いただければありがたいです。 |
thmz337
force-pushed
the
chore/replace-following-vue-component
branch
from
5929b7c to
52c5c11
Compare
|
お疲れ様です。修正遅くなり申し訳ございませんでした。 |
3 participants
Issue
概要
ユーザーのフォロー表示に関わっていた不要なVueファイルを削除した。
変更確認方法
chore/replace-following-vue-componentブランチをローカルに取り込みます。foreman start -f Procfile.devでサーバーを立ち上げます。任意のユーザーとしてログインします。
他のユーザーのプロフィールページなど、フォローボタンが表示されているページにアクセスします。
以下のフォロー/アンフォロー操作を試し、正常に動作することを確認します。
Screenshot
※ファイルの削除のみであり、UI（見た目）の変更はないためスクリーンショットはありません。
Summary by CodeRabbit