Release 2026-03-26 08:01:41 +0000#9787
Merged
komagata merged 131 commits intoproductionfrom Mar 26, 2026
Merged
Conversation
休会情報を持たない休会中ユーザーが存在していたため 新しい休会情報を作成した
休会理由を同じにしていたが、共通化する理由があるのだろうかと勘違いされる 可能性があると感じ、修正した
提出物一覧ページをReactコンポーネントからサーバーサイドレンダリングに移行。 これによりTurbo/Hotwireへの移行準備が整い、初期表示の高速化も期待できる。 - 4つのビュー(index, unchecked, self_assigned, unassigned)を更新 - 新規パーシャル(_product_list_item, _elapsed_days_header等)を追加 - ProductCheckerの初期化をVanillaJSで実装 - ヘルパーメソッドをproducts_helper.rbに追加
Metrics/MethodLength の警告を解消するため、 submitter_comment_label メソッドを抽出。
元のReact実装と同様に全件表示する。 @products と @products_grouped_by_elapsed_days の不整合を解消。
ceil ではなく floor + 1 を使用して次の経過日数までの時間を計算。 ちょうど N 日経過した時に 0 時間と表示されるバグを修正。
他のパーシャル(_elapsed_days_nav等)と同様に定数を使用し、 締切日数の変更に対応できるようにした。
以下の機能をテスト: - 商品リストアイテムの表示(ユーザーアイコン、タイトル、メタ情報) - WIPバッジの表示 - コメント数・コメント者アイコンの表示 - 未返信マークの表示 - 合格スタンプの表示 - フィルターボタンの動作(全て/未返信) - 担当する/担当から外れるボタンの動作 - 経過日数グループの表示(unassigned) - 経過日数ナビゲーションの動作 - 次の経過日数までの時間表示 - 研修終了日の表示 - ページネーションの動作 - 空メッセージの表示
API コントローラーと同様に target パラメータに基づいて フィルタリングするように修正。 - unchecked_no_replied: UncheckedNoRepliedProductsQuery を使用 - self_assigned: not_wip スコープを追加
- unassigned_controller: Product.group_by_elapsed_days を使用 - unassigned view: 到達不能な 'next if elapsed_days == 8' を削除 - product_list_test: ページネーションテストに else 分岐を追加
- Use .positive? instead of > 0 - Use .zero? and .negative? for numeric comparisons - Use .any? instead of size > 0 - Use ruby: filter for consecutive control statements - Simplify conditional assignment in filter_buttons
- Pass show_training_end_date to /products page for trainee tests - Update pagination tests for kaminari (buttons -> links) - Fix ambiguous match in filter buttons tests - Fix not-responded test to check for submitter label
Instead of hardcoding specific elapsed day keys, iterate over the products_grouped_by_elapsed_days in sorted order to maintain consistency with the main view.
Using practice title for assertion was unreliable because multiple products can share the same practice title.
- Only show training_end_date for mentor/admin, not adviser - Format training_ends_on date using l() helper - Fix pagination test to accept both /products and /products?page=1 - Show all elapsed days in nav (including days with 0 products)
`test/system/regular_events_test.rb`の変更の詳細: `regular_events`は41個存在するので、1ページに25個表示された場合、2ページ目には16個表示されるはず。つまり、mainの既存実装が誤っている mainでテストが通過していたのは、現状はJSで実装されているため、JSが非同期で表示を切り替える前に、テストがplaceholder(8個のカードが存在する)の表示をassertしてしまい通過している
コントローラーのサイズが肥大化してしまったが、別issueで対応するため
kaminariのデフォルトpager_numberが25なので情報量が増えていないことから
Use .pagination__item.is-active instead of .pagination .current to correctly match the Kaminari partial's CSS classes.
…st-to-rails-view RegularEvents.jsxを非React化
CodeRabbitレビュー対応: GITHUB_KEY, GITHUB_SECRET, STRIPE_PUBLIC_KEY, STRIPE_SECRET_KEY, POSTMARK_API_TOKENをcloudbuild-review.yamlから ハードコード値を削除し、substitution変数経由で渡す形に変更。 GitHub Secretsに登録済み。
1. cloudbuild-review-cleanup.yaml: $PROJECT_IDをハードコードに修正 (gcloud builds submit経由では未設定のため) 2. WaitForProxy: set -euo pipefail削除、if文でリトライループが正常動作するよう修正 3. CreateDB: DROP DATABASE前にpg_terminate_backendで既存接続を強制切断 4. review-app.yml: --async submit前に同一PRの実行中ビルドをキャンセル (cancel-in-progressではGCP側ビルドが止まらない問題への対応)
watchesとfootprintsの関連付けをそれぞれモジュールに統一
…on-on-pass 研修生が提出物合格になった際、その研修生が所属する企業のアドバイザーにも通知の実装
特定のURLでリンクカードが展開されないバグを修正
…ring-reaction-btn リアクションボタンをホバーした時のユーザー名を見えるようにした
….12.2 Bump icalendar from 2.12.1 to 2.12.2
- gcloud builds describeに|| trueを追加(GitHub Actionsのデフォルトset -eで 一時的なAPI失敗時にスクリプト全体が終了していた) - ポーリング間隔を10秒→30秒に(Dockerビルド20分+なので高頻度不要) - 最大120回×30秒=60分のタイムアウト
div.pill-nav__item-linkにのみcursor: pointerが設定されていたため、 通知ベルのモーダル内のbutton.pill-nav__item-link(未読/全てタブ)で マウスオーバー時にポインターが変わらなかった。 セレクタからdiv限定を外して全要素に適用。
通知ベルのピル型タブでcursor: pointerが効かない問題を修正
Q&Aページではcomments/_commentsパーシャルを使わず 独自の回答表示をしているため、#comments.thread-comments.loaded 要素が存在しない。commentContentがnullの場合にearly returnする ガードを追加。
Q&A個別ページでのcomments.js TypeError修正
Reportはafter_save_mentionで通常のメンション通知を除外していたが、 pjord宛のメンションまで除外されてしまっていた。 Report本文でpjordがメンションされた場合はPjordRespondJobをenqueueするように変更。
…ntion 日報本文でのpjordメンションにも応答するように修正
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.