Skip to content

未使用な pager.vue を削除#9034

Merged
komagata merged 1 commit intomainfrom
chore/remove-pager.vue
Aug 27, 2025
Merged

未使用な pager.vue を削除#9034
komagata merged 1 commit intomainfrom
chore/remove-pager.vue

Conversation

@sjabcdefin
Copy link
Copy Markdown
Contributor

@sjabcdefin sjabcdefin commented Aug 15, 2025

Issue

概要

app/javascript/pager.vue がどこからも使用されていないことが確認できたため、削除しました。

削除理由

  • pager.vue は questions.vue で import されています。
  • questions.vue は application.js で import され、VueMounter にコンポーネント登録されています。
  • しかし、data-vue="Questions" を持つ要素が存在しないため、Questions コンポーネントはマウントされず、結果的に pager.vue も実行されません。
  • questions.vue の削除は PR #9048 で対応しているため、本PRでは pager.vue の削除と、コンパイル通過のために questions.vue 内の pager.vue に関するロジックの削除を行っています。

変更確認方法

  1. chore/remove-pager.vueをローカルに取り込む
  2. foreman start -f Procfile.devでサーバーを立ち上げる
  3. http://localhost:3000/に接続した際に、エラーメッセージが表示されないこと

影響範囲

  • このファイルは実際に使用されていないため、削除による影響はありません。

Summary by CodeRabbit

  • 新機能
    • なし
  • バグ修正
    • なし
  • リファクタリング
    • ページネーションUIのカスタム部品を廃止し、実装を簡素化。
    • 前へ/次へ・最初/最後ボタンの表示やスタイルに軽微な差異が生じる場合があります。
  • ドキュメント
    • なし
  • その他(チョア)
    • なし

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Aug 15, 2025

Walkthrough

app/javascript/pager.vue を削除。vuejs-paginate を包むラッパーSFC(props: initialPageNumber, pageCount, pageRange, clickHandle/メソッド: makeITag)と、そのテンプレート・ローカル登録・各種クラス/テキスト設定が一括で除去された。

Changes

Cohort / File(s) Change Summary
Vue pager 削除
app/javascript/pager.vue
SFC削除。vuejs-paginateラッパー、props(initialPageNumber/pageCount/pageRange/clickHandle)、makeITag、テンプレート設定(prev/next/first/last 等)とクラスバインディング、ローカル登録を全廃。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

Possibly related PRs

Suggested labels

雑事

Suggested reviewers

  • komagata
  • okuramasafumi

Poem

ぴょんと跳ねてページは消えた
風のように軽い足取りで
わたしはうさぎ、耳で整える差分の音
余白が増えて、月もにっこり
さあ次へ—コードの道はすっきりだ 🐇✨

Tip

🔌 Remote MCP (Model Context Protocol) integration is now available!

Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats.


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 654e67a and 3a4c9a2.

📒 Files selected for processing (1)
  • app/javascript/pager.vue (0 hunks)
💤 Files with no reviewable changes (1)
  • app/javascript/pager.vue
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build_and_test
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-pager.vue

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.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@sjabcdefin sjabcdefin self-assigned this Aug 15, 2025
@sjabcdefin
Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Aug 15, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@sjabcdefin sjabcdefin force-pushed the chore/remove-pager.vue branch 3 times, most recently from 0f35c61 to 654e67a Compare August 21, 2025 11:05
@sjabcdefin sjabcdefin marked this pull request as ready for review August 21, 2025 11:33
@github-actions github-actions bot requested a review from okuramasafumi August 21, 2025 11:33
@sjabcdefin sjabcdefin removed the request for review from okuramasafumi August 21, 2025 11:34
@sjabcdefin
Copy link
Copy Markdown
Contributor Author

@ryufuta
お疲れ様です!
お手すきの際に、こちらのPRをレビューいただけると嬉しいです。😊🙏
ご負担になりそうでしたら、遠慮なく教えていただければと思います!😌👌

@sjabcdefin sjabcdefin requested a review from ryufuta August 21, 2025 11:41
@ryufuta
Copy link
Copy Markdown
Contributor

ryufuta commented Aug 22, 2025

@sjabcdefin
レビュー依頼ありがとうございます!
Vue.jsは未経験で、かつ私が担当中のIssueに関連したバグが数件見つかり立て込んでいるので、1~2週間ほどお時間いただくかもしれません🙏
それで問題なければ私がレビューをお引き受けしたいと思います。

@sjabcdefin
Copy link
Copy Markdown
Contributor Author

@ryufuta
お疲れさまです!
ご返信遅くなり申し訳ございません。🙇‍♀️

お忙しい中、レビューをご検討いただきありがとうございます。
1〜2週間ほどかかっても問題ございませんので、ぜひよろしくお願いいたします。😌🙏

Copy link
Copy Markdown
Contributor

@ryufuta ryufuta left a comment

Choose a reason for hiding this comment

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

@sjabcdefin
遅くなってすみません。確認しました。
問題ないようなのでApproveします🙆🏻‍♂️

削除理由が丁寧に書かれていたのでロジックを追いやすくて助かりました🙏
動作確認方法が書かれていませんが、一応私の方でQ&A一覧ページの表示に問題ないことを確認できています。
(descriptionに記載しているとより丁寧かと思いますが、今からわざわざ追記しなくても大丈夫だと思います👍)

ご存知がもしれませんが、Vue.jsの削除を駒形さんが急いでいるようで、#9095 で既に本PRの内容を含む形で修正が実施されています。
どちらのPRを先にマージするか駒形さんに確認するとよろしいかと思います。

@sjabcdefin
Copy link
Copy Markdown
Contributor Author

@ryufuta
お疲れさまです。😌
お忙しい中、ご確認いただきありがとうございました!

動作確認方法が書かれていませんが、一応私の方でQ&A一覧ページの表示に問題ないことを確認できています。

失礼いたしました!😮😵
Description に追加後に駒形さんにレビュー依頼しようと思います!

どちらのPRを先にマージするか駒形さんに確認するとよろしいかと思います。

教えていただきありがとうございます!
レビュー依頼時に確認しようと思います。😌👌

conflict を解消してから、レビュー依頼をする予定です。
CodeRabbit 等動くと思いますので、ご承知おきください!😌🙏

@sjabcdefin sjabcdefin force-pushed the chore/remove-pager.vue branch from 654e67a to 3a4c9a2 Compare August 27, 2025 10:33
@sjabcdefin sjabcdefin requested a review from komagata August 27, 2025 10:55
@sjabcdefin
Copy link
Copy Markdown
Contributor Author

@komagata
お疲れさまです。
お手すきの際にレビューをお願いいたします。😌🙏

1点お願いがあります。
本PRの内容が#9095に含まれています。
#9095を先にmainにマージしてしまうと、本PRが不要になってしまいますので、可能でしたら、本PRを先にmainにマージしていただけると幸いです。🙇‍♀️
ご検討お願いいたします。😌🙏

@komagata
Copy link
Copy Markdown
Member

@sjabcdefin

1点お願いがあります。

かしこまりました。

@github-project-automation github-project-automation bot moved this to 作業中 in bootcamp Aug 27, 2025
Copy link
Copy Markdown
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 89bb688 into main Aug 27, 2025
5 checks passed
@komagata komagata deleted the chore/remove-pager.vue branch August 27, 2025 12:05
@github-project-automation github-project-automation bot moved this from 作業中 to 完成 in bootcamp Aug 27, 2025
@github-actions github-actions bot mentioned this pull request Aug 27, 2025
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants