Release 2026-02-12 13:50:56 +0000#9623
Merged
komagata merged 94 commits intoproductionfrom Feb 13, 2026
Merged
Conversation
対応者と対応した日時はchecksテーブルにも記録しているためデータマイグレーション後にchecksテーブルとの関連を削除する 以下変更理由 対応済であることはinquiriesテーブルのaction_completedカラム値がtrueであることとchecksテーブルにレコードが存在することで判定していたが冗長でロジックが不必要に複雑になっていた そのため - inquiriesテーブルからaction_completedカラムを削除 - inquiriesテーブルにchecksテーブルで管理していた対応者と対応した日時のデータを移行してchecksテーブルとの関連を削除 のいずれかの対応が必要 以下の理由で今回は後者を採用した - お問い合わせに最後に誰が対応したかという情報のみ必要（未対応に戻しても変更履歴を残す必要はない）なため別テーブルで管理するメリットがない - checksテーブルは日報と提出物の確認済で利用されているがこれらとテーブル、モデル、コントローラを共通化するのはメリットが小さく複雑化するデメリットの方が大きい - お問い合わせは作成者がbootcampアプリの非ユーザー、対応者はメンターではなく管理者に限定、bootcampアプリ内で作成者と対応者の間で対応したことを通知するなどのやり取りが発生しないなど、日報や提出物とは別物なので共通点が少ない - お問い合わせを対応済みにするコンポーネントを相談部屋を対応済みにするコンポーネントで置き換え予定のため相談部屋と同様に対応と未対応の切り替えをupdateアクションで実行したい - 現状のお問い合わせの対応と未対応の切り替えはInquiriesコントローラのupdateアクションで実行しているがchecksテーブルを利用するのであればChecksコントローラのcreate/destroyアクションで実行するよう変更する
inquiriesに追加したカラムに対応済情報を記録できるようにしたためchecksテーブルを使わないようにした Inquiry.lockの部分は元々トランザクション内で実行しなければ意味がなく悲観的ロックの必要もないため削除 同時に以下のバグも修正 - 異なるユーザーが未対応のお問い合わせページを同時に表示している場合にいずれのユーザーも対応済ボタンをクリックできchecksテーブルに対応者が重複登録される - 対応者のアカウントを削除するとその対応者が対応したお問い合わせが未対応になる
checksテーブルに依存していたときとは異なり対応者アカウントが削除されても対応済のままになる
これによりcheckableに対してuserを必ず呼び出せるようになったためCheck#receiverの実装も変更
対応済みにするボタンが表示されるページには管理者しかアクセスできないが、APIは管理者以外も叩けるようになっていた
CodeRabbitの指摘に対応：#9423 (comment)
CodeRabbitの指摘への対応 #9423 (comment) #9423 (comment)
moreCommentButtonとmoreCommentsの取得が3箇所に 重複していたのをトップレベルで1回だけ取得するように整理した。
Capybara.reset_sessions!がNet::ReadTimeoutで失敗した際に エラーを握りつぶすだけではセッションが壊れたまま残り、 後続のvisitもタイムアウトしてフレーキーテストの原因になっていた。 page.driver.quitでブラウザを終了させ、次のテスト操作時に Capybaraが新しいブラウザを自動起動するようにした。
github-actions
bot
changed the title
…request_js fetcher.jsを廃止しrequest.jsに処理を統一
github-actions
bot
changed the title
RuboCop拡張の読み込み方法をプラグイン方式に移行した
github-actions
bot
changed the title
…on-for-inquiry お問い合わせを対応済にする機能のリファクタリングと軽微なバグ修正・仕様変更
github-actions
bot
changed the title
…-mentioned-in-micro-report 分報にメンション機能を追加
github-actions
bot
changed the title
Bumps [faraday](https://github.com/lostisland/faraday) from 2.14.0 to 2.14.1. - [Release notes](https://github.com/lostisland/faraday/releases) - [Changelog](https://github.com/lostisland/faraday/blob/main/CHANGELOG.md) - [Commits](lostisland/faraday@v2.14.0...v2.14.1) --- updated-dependencies: - dependency-name: faraday dependency-version: 2.14.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bump faraday from 2.14.0 to 2.14.1
github-actions
bot
changed the title
- disable_ddl_transaction! を追加することで自動トランザクションを意図的にオフにしている - 詳細は以下を参照してください - https://railsguides.jp/active_record_migrations.html#トランザクション
assert_differenceブロック内でclick_button後にDB書き込み完了を 待たずカウントを確認していたため、タイミングによってEvent.count が変化しないことがあった。ブロック内でフラッシュメッセージの 表示を待つことでDB書き込み完了後にカウントを確認するようにした。
…ents fix: 折り畳まれたコメントへのアンカーリンクが効くようにする
github-actions
bot
changed the title
fix: イベント作成テストのフレーキーを修正する
github-actions
bot
changed the title
…-in-discord-notifications Discordのみんなの日報に「今日の気分」の絵文字を載せる
github-actions
bot
changed the title
fix: vectorのextensionを有効にするマイグレーション時にエラーが発生してもスルーされるようにした
github-actions
bot
changed the title
コピーしたメッセージの背景色が出ていないのを修正
github-actions
bot
changed the title
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
7 participants
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.