[プラクティス編集]プラクティス、コース、カテゴリーの編集権限を管理者からメンターに移行。

[machida]

現在、プラクティス、コース、カテゴリーの編集権限は管理者しか持っていない。
これを管理者とメンターに変更をしたい。
変更に伴い URL も変更したい。

現在は、

• /admin/categories
• /admin/courses
• /admin/practices

となっている。

これを

• /mentor/categories
• /mentor/courses
• /mentor/practices

にし、プラクティス、コース、カテゴリーのタブをメンターページのタブに移す。

[mono-nobe]

@machida
改修方針について2点確認させてください！

管理ページのタブは削除してもよろしいですか？

issueの概要に

プラクティス、コース、カテゴリーのタブをメンターページのタブに移す。

とあるのですが、管理ページからタブ自体削除してもよろしいですか？

クラス(CSS)もメンターページ用に置き換える必要はありますか？

ページを置き換える際のクラスの扱いについて、以下のどちらが良さそうですか？
(他に良い方針などあればご教示お願いします🙇 )

• mentor用のページを作成する際、置き換え対象のページに使用されている各クラス名をmentor用に置き換える
  • 具体的には、mentor用のスタイルシートを新たに作成し、admin-table__item-value クラスを mentor-table__item-valueとする
  • => 差分がかなり大きくなりそう
  • 仮にクラス名も置き換える場合、他ページに影響がないことを確認した後、admin用のスタイルシートを削除してもよろしいですか？
• 既存のクラスを参照する
  • 具体的には、コンポーネント名を MentorCourse とし、クラス名で admin-table__item を使用する
  • => 置き換え後のコンポーネント名とクラス名に差が生じるため誤解を招く可能性がある

対象ページ

プラクティス

カテゴリー

コース

[machida]

@mono-nobe

質問ありがとうございます！！

管理ページのタブは削除してもよろしいですか？

はい！削除でお願いしますー。

クラス(CSS)もメンターページ用に置き換える必要はありますか？

これは置き換えずにこのままでお願いします。
別Issueで、メンター用のクラス名を付けるのではなく、admin-table の admin をもっと抽象化して admin にも mentor にも使えるように management-table などにする予定ですー

[mono-nobe]

@machida
ご回答ありがとうございます！
上記、承知しました。いただいた内容で進めます！

[mono-nobe]

@machida
権限周りについて、各項目の作成はこれまで通り 管理者のみ行える 認識であっていますか？
その場合、管理者の routes や controllers などには作成に必要な処理のみ残し、他は削除してもよろしいですか？

(追記)
プラクティスの作成は元からメンターも行える(ただし、編集はできない)ようですが、こちらについては作成できるままでよろしいですか？
また、プラクティスの順番の変更はメンターもできるようにしてもよろしいですか？

[mono-nobe]

@machida

別Issueで、メンター用のクラス名を付けるのではなく、admin-table の admin をもっと抽象化して admin にも mentor にも使えるように management-table などにする予定ですー

ファイルやディレクトリ、変数を命名する際には、mentor の代わりに management を使用した方がよろしいですか？
management を使用することのメリットは同上で、デメリットは、既存のファイルや変数名などにmentorが使用されており、それらと混在し煩雑になることです。
個人的には、影響範囲が広いこともあり、mentorを使用する方が良い気がします。
(URLも /mentor/categories のようにするため、 mentor で良さそう...？)

[mono-nobe]

@machida

カテゴリー削除の権限はメンターに移行しますか？ 👀
(編集画面にも削除ボタンがあることを確認しました)

[mono-nobe]

権限周りについて、各項目の作成はこれまで通り 管理者のみ行える 認識であっていますか？

作成や削除など諸々含め移行する

[mono-nobe]

ファイルやディレクトリ、変数を命名する際には、mentor の代わりに management を使用した方がよろしいですか？

mentor を使用する

[machida]

@mono-nobe

まとめておきますー

📝

権限周りについて、各項目の作成はこれまで通り 管理者のみ行える 認識であっていますか？

メンターにも管理者同様の権限の追加をお願いします。
プラクティス周りの権限はメンターと管理者が持つようにお願いします。

ファイルやディレクトリ、変数を命名する際には、mentor の代わりに management を使用した方がよろしいですか？

mentor でお願いします。
管理画面に使っている table の HTML の class 名だけ、変更する予定です。

カテゴリー削除の権限はメンターに移行しますか？ 👀

こちらの権限も他のプラクティス周りと同様にメンターと管理者にお願いします。

[mono-nobe]

📝 2023/03/29 振り返りMTGメモ
PRの差分が多いが、単純な置き換えであるため1つのPRで対応する。

[github-actions]

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

[mono-nobe]

@komagata @machida
本番環境へリリースされましたので、お手隙の際に動作確認をお願いします 🙇

[komagata]

@mono-nobe 何を確認すればいいのかの指示をお願いします〜

[mono-nobe]

@komagata
失礼しました... 🙇
管理者権限を持つ駒形さんにはプラクティス、コース、カテゴリーのタブが管理者ページに存在しないことを確認していただきたいです！(以下PRの8番以降になります)
#6344 (comment)

また、別途メンター権限のみ持つ方へ動作確認依頼しようと思うのですが、コースやカテゴリーの作成なども実行いただいて良いのでしょうか。。？
(本番環境で使用しないコースなどが作成されるのは良くないかと思い)

[komagata]

@mono-nobe

管理者権限を持つ駒形さんにはプラクティス、コース、カテゴリーのタブが管理者ページに存在しないことを確認していただきたいです！

こちら確認致しました。問題ありませんでした。

また、別途メンター権限のみ持つ方へ動作確認依頼しようと思うのですが、コースやカテゴリーの作成なども実行いただいて良いのでしょうか。。？

はい、その後削除していただけるのであれば問題ありません。

[mono-nobe]

@komagata
すみません、、こちら見逃しておりました 🙇

はい、その後削除していただけるのであれば問題ありません。

承知しました！
こちらの内容でメンターの方に依頼しようと思います。

[mono-nobe]

@fjordllc/mentor
お手数ですが、移行した以下のタブの機能が使えるかご確認いただきたいです。 🙇

• プラクティス
• コース
• カテゴリー

具体的な確認項目については、
以下PRの概要に記載されている 変更確認方法 の 3.ヘッダーからメンターページに移動する から 7.コースページについて、以下の内容を確認する となります。

#6344 (comment)

[mono-nobe]

@cafedomancer

お手数ですが、移行した以下のタブの機能が使えるかご確認いただきたいです。 🙇

ご対応いただきありがとうございました！
とても助かりました。 😊

[mono-nobe]

本Issueでの確認は完了しましたのでcloseします。
https://discord.com/channels/715806612824260640/809595476847493192/1140632509802098768