Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. 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. Comment |
|
@matuaya ブックマークのラベル部分について、現状の実装だと「定期イベント」などの長いモデル名が上手く折り返しができないと思うので、その部分を対応していただきたいな〜という話です。 以下が参考のPRです 具体的には bootcamp/app/components/dashboard_bookmark_component.html.slim Lines 8 to 10 in bf3d65b の type.humanのところを私の方のPRで作成したLabelHelperを使ってbookmarkable_label(type)みたいにしていただければ上手く折り返しができるのではないかと思います。
ご検討の程をお願いします! |
|
@tyrrell-IH |
|
@matuaya |


Issue
- BookmarksInDashboard.jsxを非React化する #9045
概要
ダッシュボードのブックマーク表示機能をReactからViewComponentに移行しました。
変更確認方法
chore/replace-react-with-viewcomponent-for-dashboard-bookmarksをローカルに取り込むScreenshot
変更前
変更後