コースに除外するプラクティスを設定できるようにする

[komagata]

コースは現在、カテゴリー単位でプラクティスを設定できるようになっている。
しかし、プラクティス単位で除外（飛ばしてOK）にしたい。

画面表示としては、プラクティス一覧でそのプラクティスが灰色になっている感じ。

細部の仕様については応相談。

[machida]

@kyokucho1989

#7483
こちらのPRを手元にpullして作業をしてください。
管理画面側の見た目だけ実装してあります。

仕様

管理画面側

メンターでログインし、コースの作成画面 or 編集画面にアクセス。
/mentor/courses/528030687/edit

---

コースは、そのコースに含めるカテゴリーを選択することで作成できる。

• コースはカテゴリーのグループである。
• カテゴリーはプラクティスのグループである。

---

コースに含めるカテゴリーを選択。

すると、 カテゴリーが開き、そのカテゴリーに所属するプラクティスが全て表示される。

その中から、スキップするプラクティスを選択する。

ユーザー側

スキップになったプラクティスはこんな感じで色が変わるようにする（デザインはmachidaがやります）

色は変わっているが、プラクティスのshowにアクセスはできる。

でも、スキップをするプラクティスだとわかるようになっている。

[kyokucho1989]

@machida
ありがとうございます。仕様について質問があります。

現状、プラクティスは複数のカテゴリに所属させることが可能です。その場合、プラクティスの表示画面が不適当になる場合があります。

例：

1. プラクティスaをカテゴリAとカテゴリBに所属させる
2. スキップするプラクティスを設定
3. カテゴリAに所属するプラクティスaはスキップ
4. カテゴリBに所属するプラクティスaはスキップしない
5. プラクティスページを表示させると。。。矛盾する？

質問
この場合はどう実装すればいいでしょうか。以下、案を書きます。

案1: スキップさせるチェックボックスをプラクティスごとに同期させる
　上の例で言えば、カテゴリAのプラクティスaのチェックボックスをONにすると、
　カテゴリBのプラクティスaもONになる、というイメージです。

案2: プラクティスが所属するカテゴリはひとつだけになるようバリデーションを付与
　選択はチェックボックスではなくラジオボタンにする

複数のカテゴリに所属できるようにする、というのは以下のPRにあります。

#1855

が、経緯がいまひとつ分かりません。問題なければバリデーションを付与し、案2の方針で実装したいです。

以上です。よろしくお願いします。

[machida]

@kyokucho1989
あー確かに！
まずは、経緯を説明します。

今回の機能は研修生用のものになります。研修生は時間制限があるので、全てのプラクティスをやらずいくつかのカテゴリーを選択したコースを作成し、そのコースで学習します。しかし、コースにはカテゴリーを選択することしかできず、カテゴリーに含まれているこのプラクティスは抜かしたい、ということができません。例えば、この研修生はRubyを大学で学習したからできるんだけど、復習を兼ねてRubyの後半だけやらせたい、みたいなことが稀にあります。ここで必要になったのがこの機能になります。

という訳で、「このプラクティスはスキップする」というデータはカテゴリーではなく、コースに持たせる必要があります。コースAはプラクティスZをスキップするが、コースBはプラクティスZをスキップしない、ということに対応できる必要があります。

プラクティス一覧（つまりコースページ）では、それは可能だと思いますが、kyokuchoさんの指摘どおり、どのコースもプラクティスshowは同じページを見ているので、リンク元によって表示を変えることができないですね。なので、今回は一覧のときはグレイにするが、showでの表示は同じにしたいと思います。

もしくは、少し手をかけて、グレイになっている（スキップする）プラクティスだけリンク先のURLにパラメータを付け、そのパラメータのある・無しでshowの表示を分ける、というのはできないかな、と思いました。

---

この機能はコースではなくユーザーに持たせた方がいいかも？と思い始めて来ました。スキップするプラクティス情報はコースに持たせるのではなく、ユーザーに持たせたほうがいいかもしれないです。
これについて今日確認するので、決まり次第連絡をします🙏

[machida]

@kyokucho1989

お待たせしました！
すいません、スキップするプラクティスはユーザーに紐づけるで確定になりました。

以下のようにユーザーの編集画面で、メンターと管理者だけスキップするプラクティスが設定できるようにしたいです。
#7483
こちらのプラクティスのデザインを修正します。

---

また、どのプラクティスをスキップしているかを

提出物個別ページのユーザーメモの下に表示したいです。
こちらもメンターと管理者だけが見れるようにお願いします。

[machida]

@kyokucho1989

管理者でログインし、ユーザー kenshu でログインをすると手元デザインが確認できるようにしました。

[kyokucho1989]

ありがとうございます！
家に帰って諸々確認します

[machida]

よろしくお願いします🙏

[kyokucho1989]

@machida
おはようございます！　2点確認させてください。

1. 研修生から見た画面の変更
　「研修生からみたデザイン変更はプラクティス一覧画面のみで、プラクティス詳細画面は変更なし」という認識で良いでしょうか？

2. 進捗率の計算方法
　スキップしたプラクティスは「修了したプラクティス」の分母に含まないように計算する必要がありますか？

というのも、プラクティスが複数のカテゴリに所属を可能にした状態で、「プラクティスaについてカテゴリAはスキップOK、カテゴリBはスキップNG」とすると、計算が合わなくなりそうだからです。

以上です！！

[machida]

@kyokucho1989 質問ありがとうございます！
プラクティス個別ページにも、着手とかのステータスのボタンを非表示にして、このプラクティスはスキップするとわかるような文字を表示しようと思います。分岐を作っておいてください🙏デザインはやります。

進捗バーは確かに計算ずれますね。こちらも対応いただけると助かります🙏

[kyokucho1989]

@komagata @machida
お疲れ様です。こちら、ユーザー登録情報ページでスキップするプラクティスの設定をするところまでは実装が完了しました。
ただ、プラクティス一覧ページの修正は #7527 で非Vue化を進めています。
これがマージされ次第進めていこうと思います。

また、進捗バーの修正については下記のバグが報告されています。
#7635
修了したプラクティスの数を正してから、進捗バーの計算を行おうと思います。

#7635 を自分に割り当てていただけませんか？

[machida]

@kyokucho1989 了解です！アサインさせていただきますー

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

[github-actions]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。