-
Notifications
You must be signed in to change notification settings - Fork 75
ペアワークをサポートする機能を実装 #9626
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
ペアワークをサポートする機能を実装 #9626
Changes from all commits
Commits
Show all changes
209 commits
Select commit
Hold shift + click to select a range
37bf984
ペアワーク機能を実装
mousu-a e315c0c
ペアワークに通知機能を追加
mousu-a 2281d60
ペアワークの通知のテストを追加
mousu-a efbf49c
関係者の近日開催イベント欄にペアワークを表示するように
mousu-a e2032fb
動作確認用のフィクスチャを作成
mousu-a 8c37929
ペアワークのチャンネル選択のカラムを追加
mousu-a d0057ba
ペアワークのチャンネル選択に関するViewの記述を追加
mousu-a 65f7c39
ペアワークのコメントが0で相手が見つかっていないとき、Q&Aのように"コメント"を赤文字で表示する
mousu-a 0051867
ペアワークのヘルパーテストを導入
mousu-a 84a4a17
近日開催するペアワークのヘルパーメソッドテストを導入
mousu-a fd4ee2d
ペアワークのモデルテストを導入
mousu-a 12a3ace
Rubocopの指摘点を修正
mousu-a 2760f4b
edit画面ではscheduleを表示しないようにした
mousu-a 84bed91
重複を削除 頭痛が痛いみたいになっていたので
mousu-a 42ad5f3
不要な分岐だったので
mousu-a 5caf048
レイアウト崩れの修正
machida 7dbe7d0
ペアワークの相手をペアに名称変更、確定ブロックのベースを作成
machida 093e775
デザイン更新
machida 6b2722a
ペアワークをQ&Aとセットにし、デザインを更新した
machida b8d7112
ペアワークのシステムテストを実装
mousu-a 8d84fcc
Rubocopの指摘点を修正
mousu-a 6baedfd
関連付けを使用
mousu-a 372850b
ペアワークのdesctiptionにNotNullを追加
mousu-a 24a9156
ペアワークを本番で非表示にした
machida 190397f
:cop:
machida dedb399
ペアワークリリース後のTODOコメント追加
machida 9a434bc
pair_workのsave時に関連付けを利用してschedulesもsaveするように
mousu-a 8bf6532
Watch機能の非Vue化に対応
mousu-a 7c99f2a
_formのリファクタ
mousu-a 8881000
希望日時(schedules)が空の状態ではペアワークのsaveが失敗するように
mousu-a 6ed9c7e
一応proposed_atにpresence trueをつけておく
mousu-a d80790e
Rubocopによる指摘点を修正
mousu-a 880fb2c
テストが通らなくなっていたため
mousu-a 6e079e5
newspaperで実装していた部分を置き換え
mousu-a 0b5067e
wip作成時の挙動をQ&Aと同じように
mousu-a 4cca439
Q&Aのコメント欄が表示されなくなっていたため
mousu-a bc0224f
フラッシュメッセージを共通化
mousu-a ed1a676
disabledをメソッドで共通化
mousu-a 6d12880
変数名をわかりやすく
mousu-a 0449de4
不要な記述だったため
mousu-a fd0a94f
不要な変更だったため
mousu-a 8eae7ed
心配なのでシステムテストを追加
mousu-a c787eef
Rubyのメソッドを使わずSQLでフィルタリングする
mousu-a bab939d
誤字修正
mousu-a a38de74
近日開催されるペアワークのテストをわかりやすく、且つ順序に依存しないように修正
mousu-a 4fc0e6a
誤って表示していたと思われるリンクを削除
mousu-a 89f9db0
ペアワークのレコードがない時の表示が機能していなかったので修正
mousu-a 5c5e2c2
既存の実装に合わせてunsolved_badgeを使用
mousu-a c112bf2
クエリを無駄に繰り返していたので一回にまとめた
mousu-a 544386d
WIP 近日開催のペアワークを仕様変更
mousu-a c24222d
挙動をチェックしやすいようにフィクスチャを追加
mousu-a 2c33bcd
ダッシュボードのペアワークの表示を整えた
machida df67ed2
必要なさそうだったのでPresenterの導入をやめた
mousu-a df18946
必要なくなったため削除
mousu-a 3a97a4e
Rubyの記法にならって変更
mousu-a 8bba87c
モデルテストを追加
mousu-a efda073
変更が不要になったため戻した
mousu-a 040ac21
PRに不要なdiffだったため
mousu-a 08c0dda
update, destroyで権限チェックをするように
mousu-a ab64a51
コメント消し忘れ
mousu-a acd0270
誤字修正
mousu-a 5ccfe7c
コンフリクト解消でテストが落ちていたので修正
mousu-a 79e6874
修正漏れ unsolved_badgeを使うように
mousu-a 5ac4c44
check_box_tagを使うことで不要なhiddenフィールドが生成されないように
mousu-a f0d97ff
変数名、メソッド名をわかりやすく
mousu-a 950efa8
Rubocopによる指摘点を修正
mousu-a e1326cf
admin以外のメンターがマッチングできない問題を修正
mousu-a 87f4341
adminではなくメンターでテストするように
mousu-a 76bcf9d
不要な記述を修正
mousu-a d84cdc8
wipにデフォルト値を、検索の高速化としてpublished_atにindexを設定
mousu-a e45ca3e
作成者とadminだけが編集できるように
mousu-a 6714dfe
Rubocopによる指摘点を修正
mousu-a 0ac5acf
テストが通るように
mousu-a e0da709
わかりやすい変数名に
mousu-a 39d4a90
関連付けを使用して取得するように
mousu-a 38f4eb1
将来の、ペアワークのスケジュールの変更機能のためallow_destroy: trueをつける
mousu-a 092bc98
if_not_exists: trueをつけて安全に
mousu-a d609ba1
想定していない日時でのマッチングを拒否するバリデーションを実装
mousu-a 3651d53
どのuserを参照しているかがわかりにくいため
mousu-a 954e3f9
inverse_ofをつけて明示的に
mousu-a 3918543
pair_work.userでbuddyの値が返ってきてしまっていたため修正
mousu-a 8b4d01b
DRYにした
mousu-a f710eb7
partialをcollection形式にした
mousu-a 9e5fbcf
ヘルパーメソッドのテストを追加
mousu-a 94df9a7
不要だったため
mousu-a d2eb7fd
汎用性が高いpair_workをフィクスチャで生成するようにした
mousu-a 0afa0d5
メソッドをよりわかりやすくした
mousu-a 89facdb
nilの確認だけで良さそうだったので修正
mousu-a 5a53ca4
メソッドをよりわかりやすくした
mousu-a b435c5b
ペアワーク作成者の場合の表示を変更、partialを導入し整理した
mousu-a ad2d139
メソッド名の変更 紛らわしかったため
mousu-a 2d81de6
ヘルパーメソッドのテストを追加
mousu-a 2665cb8
ユーザー退会時、未実施のペアワークを削除するようにした
mousu-a 2b886a0
検索機能の実装変更に対応
mousu-a 2797d05
コメントが機能していなかったので修正
mousu-a 629801c
テストが落ちるようになったため修正
mousu-a 43224b6
検索機能の変更に伴い、テストが落ちてしまっていたので
mousu-a 1488c5a
メソッド修正 buddy_idさえあればペアワークの内容修正が出来てしまうのを修正
mousu-a c7e2486
ペア確定後、スケジュール表が見えなくなっていたので修正
mousu-a 25b907d
pair_work_user?を削除、意味がなく冗長だったため
mousu-a 00e9f93
実装変更に伴いテストを修正
mousu-a d15850d
Rubocopによる指摘点、誤字を修正
mousu-a b549230
検索機能の変更に伴い、テストが落ちてしまっていたので修正
mousu-a b714313
誤字修正
mousu-a 9b1bc11
メンターにのみ削除ボタンが表示されるようになっていたので修正
mousu-a 5712bf9
重複を避けるため変数を定義
mousu-a 2691bbf
メソッド修正 permissionチェックをまとめた
mousu-a 4cc263b
ダッシュボードのアイコン変更、左ナビのサイズ微調整
machida 3f2ea87
PairWorkモデルのsolved?とimportant?メソッドのテストを追加
machida 569ebb3
ペアワークのdatetime属性をISO8601形式に変更
machida 0bacfed
upcoming_pair_worksのテストを時間固定で実行するよう修正
machida 4078511
PairWorkテストのbuddy_idをbuddyアソシエーションに変更
machida b4f32ee
テスト まとめて良さそうだったので
mousu-a 8a3916e
テスト名を修正
mousu-a a32f304
ペアワークの未解決数をキャッシュ化してパフォーマンスを改善
machida 0292b6b
インデントを修正
mousu-a 1b29f7d
マッチング時の確認をより厳格に それに伴いテストも追加
mousu-a 10217d0
ペアワーク作成者がメンターだった場合に重複しないように
mousu-a 594231a
他の実装と合わせ1行開けた
mousu-a 3bab488
不要な記述を削除
mousu-a 5520e52
Viewのロジックをhelperに移譲
mousu-a 5e29efc
HTMLタグ 短いものは1行に
mousu-a 16ba5c8
schedules 名前が衝突しそうなのでpair_work_schedulesに変更
mousu-a 4ede13c
メソッド修正 命名が抽象的すぎたため
mousu-a b2fee70
変数名の修正 実態に即した形に
mousu-a f17e3db
タイポ修正
mousu-a 78d5d4a
schedulesテーブルのリネームでCIが落ちていたので修正
mousu-a 2595ce5
既存の実装と合わせ1行開ける
mousu-a be974b9
ペアワークの通知が重複しているため、Callbackでの通知を削除
mousu-a 5dc6b19
upcoming_pair_works、not_held に当日の開催済みペアワークが入らないように
mousu-a a02e351
変数名の修正 実態に即した形に
mousu-a 966db77
無駄に繰り返していたため事前に計算する形に
mousu-a 597a45c
不要なインデントを削除
machida 52d09ca
Rails 7.2 / Ransack 4互換性のため
mousu-a d7b7eaf
テスト修正 当日の開催済みのペアワークは含めないように
mousu-a 87bb1be
変数修正 より実態に即した形に
mousu-a deb0986
テスト修正 型の不一致を避けるため
mousu-a 43e2158
テスト修正 よりわかりやすく
mousu-a 5b4c887
テスト修正 他の人が疑問を抱かないようtravel_toで囲んでおく
mousu-a bb6a222
rebaseミスで混入したdiffを削除
mousu-a e5ebec4
edit(編集画面)が誰でも見られるようになっていたので修正
mousu-a fde2146
before_actionで定義する意味がなさそうだったので修正
mousu-a 1b7435a
必要な関連をpreloadするように修正
mousu-a 982cbc2
includesに変更 明示してpreloadにする意味がなさそうだったので
mousu-a 0a210b0
エラーが起きてしまっていたため修正
mousu-a 27f9773
preloadする関連を増やした
mousu-a 91461e4
Q&Aの実装と合わせ、showではpreloadしない形に変更
mousu-a cdeeded
タイポ修正
mousu-a 1d1ea86
ペアワークのスケジュール日時にユニーク制約をつけた
mousu-a f210d01
Railsの慣習に従い記述を変更
mousu-a 2328f5e
不要なファイルを削除
machida 89fc9dd
未実装の機能のリンクにTODOコメント追加
machida a7010a8
time周りのマークアップミスの修正
machida df73d25
Railsの慣習に従い記述を変更
mousu-a 758aa5c
無駄なクエリを走らせていたため修正
mousu-a 402c66d
フィクスチャ修正 それぞれの役割をわかりやすく
mousu-a f28b885
アソシエーション名を指定するように
mousu-a 6d983a3
CSSの重複を削除
machida 8f93d9d
記法を統一
mousu-a 59a1552
メソッド修正、合わせてテストも修正
mousu-a dd636fc
変数名修正 わかりやすく
mousu-a 0a77e2b
before_actionで定義する意味がなさそうだったため
mousu-a d5088d4
mentioner ペアワークのコメント欄でメンションされた時に正しく表示されるように
mousu-a bceec0f
不要なdiff
mousu-a 50231cb
冗長なため1行に
mousu-a 80bce54
記法を統一
mousu-a 50024d1
メソッド名修正 わかりやすく
mousu-a 58cb820
文言を統一
mousu-a 4394e96
ペアワークの通知テストを分割
mousu-a 08e823a
Tailwindユーティリティクラスにimportantを付与、pair-workのCSS読み込みを追加
machida 1b4b369
チャンネルのデフォルト値をafter_initializeで入れるようにした
mousu-a aecf1e8
インデントの不整合を修正
mousu-a 9668fd1
検索用メソッドの重複を削除
mousu-a 2d74a33
ペアワーク機能の表示制御をフィーチャーフラグに変更
mousu-a f3c6b5f
ペアワークのページにいるとき、Q&Aとペアワークのグローバルナビがアクティブになるように修正
mousu-a eb3307a
デフォルトのチャンネルのセットを明示的に
mousu-a f9f74c5
PairWorkのmatching処理を別コントローラに切り出した
mousu-a bb317dc
メソッド名修正 勘違いしそうだったため
mousu-a 34b9225
matchではなくreservationに統一
mousu-a c72c6bd
Rubocopによる指摘点を修正
mousu-a fb05fa9
Viewでしか使われていなかったメソッドをヘルパーに移譲
mousu-a 84ca010
マッチング時の挙動を正しくテストするために修正
mousu-a 2731106
不要そうだったため
mousu-a bb94713
マッチング時のバリデーション 存在チェックも追加
mousu-a 0f293ee
不要だったため
mousu-a 0fc3038
Watchの一斉createがペアワークの公開時にのみ作動するように
mousu-a e89c777
マイグレーションファイルを現在のRailsバージョンに統一
mousu-a 5ff6563
マッチング失敗時にコメントが表示出来ずエラーになってしまうため修正
mousu-a e466e2b
fix: 未参加イベントのヘッダー色を親要素と統一する
machida 37258dd
fix: ペアワークのチェックマークにcursor: pointerを追加する
machida 48a0d37
マッチング時のbuddyをコントローラで入れる形に
mousu-a d407d99
マッチング失敗時にパーシャルを明示的に指定する
mousu-a 223c082
マッチング失敗時のエラーメッセージを表示するように
mousu-a 3142584
マッチング時のバリデーションをカバーするテストを追加
mousu-a f8f09fc
エラーメッセージの文言を統一
mousu-a 03a5fd9
不要になったバリデ、テストを削除(buddyをコントローラで入れるようにしたため)
mousu-a 2a20e0b
Rubocopによる指摘点を修正
mousu-a 2e5a081
テストがflakyだったため待つためのコードを追加
mousu-a 2c22ddf
冗長だったため
mousu-a 121d6d8
慣習に合わせファイル名を複数形に
mousu-a 94550a5
important?メソッドをDecoratorに移譲
mousu-a da806ba
メソッド修正 Helperメソッドはスコープが広いためより具体的に
mousu-a 295923a
メソッド修正 _scheduleと_formで共通化できるようにした
mousu-a 0040b53
慣習に合わせ変更
mousu-a 5e75c6a
フィクスチャの修正
mousu-a ed862a9
Viewのロジックをヘルパーメソッドに
mousu-a 0315198
記法を統一
mousu-a 8ca16f1
テスト、フィクスチャ修正
mousu-a 50b0509
テスト修正 他のコードに合わせて空行を追加
mousu-a File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
13 changes: 13 additions & 0 deletions
13
app/assets/stylesheets/application/blocks/page/_page-body-header.css
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| .page-body-header__inner { | ||
| display: flex; | ||
| align-items: center; | ||
| gap: 1rem; | ||
| margin-bottom: 1rem; | ||
| } | ||
|
|
||
| .page-body-header__title { | ||
| font-size: 1rem; | ||
| line-height: 1.4; | ||
| font-weight: 700; | ||
| color: var(--main); | ||
| } |
48 changes: 48 additions & 0 deletions
48
app/assets/stylesheets/application/blocks/pair-work/_pair-work-info.css
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,48 @@ | ||
| .pair-work-info { | ||
| display: flex; | ||
| gap: 0.75rem; | ||
| } | ||
|
|
||
| .pair-work-info__user-icon { | ||
| width: 3.5rem; | ||
| height: 3.5rem; | ||
| } | ||
|
|
||
| .pair-work-info__end { | ||
| flex: 1; | ||
| } | ||
|
|
||
| .pair-badge { | ||
| font-family: serif; | ||
| border: double 0.1875rem var(--stamp-color); | ||
| border-radius: 0.75rem; | ||
| width: 4rem; | ||
| height: 3.5rem; | ||
| padding: 0.125rem; | ||
| position: absolute; | ||
| z-index: 1; | ||
| transform: rotate(25deg); | ||
| font-size: 1.125rem; | ||
| line-height: 1.1; | ||
| display: flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| font-weight: 800; | ||
| color: var(--stamp-color); | ||
| flex-direction: column; | ||
| pointer-events: none; | ||
| } | ||
|
|
||
| @media (min-width: 48em) { | ||
| .pair-badge { | ||
| right: 0; | ||
| top: -0.125rem; | ||
| } | ||
| } | ||
|
|
||
| @media (max-width: 47.9375em) { | ||
| .pair-badge { | ||
| left: 80%; | ||
| top: 0; | ||
| } | ||
| } |
44 changes: 44 additions & 0 deletions
44
app/assets/stylesheets/application/blocks/pair-work/_pair-work-schedule-dates.css
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| .pair-work-schedule-dates.is-solved .pair-work-schedule-dates__table { | ||
| display: none; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates.is-solved input:checked + .pair-work-schedule-dates__table { | ||
| display: block; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates__action-items { | ||
| display: flex; | ||
| justify-content: center; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates__action-item { | ||
| min-width: 20rem; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates__action-item-description { | ||
| margin-top: 0.25rem; | ||
| font-size: 0.75rem; | ||
| line-height: 1.4; | ||
| text-align: center; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates__title { | ||
| font-size: 1rem; | ||
| line-height: 1.4; | ||
| font-weight: 600; | ||
| margin-bottom: 1rem; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates__action + .pair-work-schedule-dates__table-container .pair-work-schedule-dates__table { | ||
| margin-top: 1rem; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates__cancel { | ||
| display: flex; | ||
| justify-content: center; | ||
| margin-bottom: 1rem; | ||
| } | ||
|
|
||
| .pair-work-schedule-dates__cancel-action { | ||
| min-width: 20rem; | ||
| } |
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
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| .page-tabs | ||
| nav.page-tabs | ||
| .container | ||
| ul.page-tabs__items | ||
| - tabs.each do |tab| | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,9 @@ | ||
| li.tab-nav__item | ||
| = link_to link, class: "tab-nav__item-link #{active ? 'is-active' : ''}" do | ||
| = name | ||
| - if count.present? | ||
| = " (#{count})" | ||
| - if badge&.positive? | ||
| .page-tabs__item-count.a-notification-count | ||
| .not-solved-count | ||
| = badge |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,15 @@ | ||
| # frozen_string_literal: true | ||
|
|
||
| class SubTabComponent < ViewComponent::Base | ||
| def initialize(name:, link:, active: false) | ||
| def initialize(name:, link:, active: false, count: nil, badge: nil) | ||
| @name = name | ||
| @link = link | ||
| @active = active | ||
| @count = count | ||
| @badge = badge | ||
| end | ||
|
|
||
| private | ||
|
|
||
| attr_reader :name, :link, :active | ||
| attr_reader :name, :link, :active, :count, :badge | ||
| end |
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| # frozen_string_literal: true | ||
|
|
||
| class PairWorks::ReservationsController < ApplicationController | ||
| before_action :require_mentor_login, only: %i[create] | ||
|
|
||
| def create | ||
| @pair_work = PairWork.find(params[:pair_work_id]) | ||
| if @pair_work.reserve(pair_work_reservation_params) | ||
| ActiveSupport::Notifications.instrument('pair_work.reserve', pair_work: @pair_work) | ||
| redirect_to Redirection.determin_url(self, @pair_work), notice: @pair_work.generate_notice_message(:reserve) | ||
| else | ||
| @comments = @pair_work.comments.order(:created_at) | ||
| render 'pair_works/show' | ||
| end | ||
| end | ||
|
|
||
| def destroy; end | ||
|
|
||
| private | ||
|
|
||
| def pair_work_reservation_params | ||
| params.require(:pair_work).permit(:reserved_at).merge(buddy_id: current_user.id) | ||
| end | ||
coderabbitai[bot] marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| end | ||
Oops, something went wrong.
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.