Skip to content
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

メンターの責務をメンター用Controllerに集約する #6536

Closed
mono-nobe opened this issue May 19, 2023 · 2 comments
Closed

メンターの責務をメンター用Controllerに集約する #6536

mono-nobe opened this issue May 19, 2023 · 2 comments
Labels
Projects

Comments

@mono-nobe
Copy link
Contributor

mono-nobe commented May 19, 2023

概要

メンターのみが使用できるメソッドの一部がメンター用Controllerに定義されていないため集約したい。
また、それに伴い不要な画面などあれば削除する。

詳細

以下のメンター用 Controller において、一般の Controller と処理が重複しているメソッドが存在する。
現状、一般の Controller に定義されているメンター用メソッドは、before_action :require_admin_or_mentor_login によりメンターのみが使用できるようになっている。

  • メンター用
    • Mentor::CoursesController
    • Mentor::PracticesController
    • Mentor::CategoriesController
  • 一般
    • CoursesController
    • PracticesController
    • Courses::CategoriesController

また、処理の重複に伴い、同じ機能を持った画面が存在している。
以下、どちらもメンターのみが閲覧できるコース一覧画面である。作成画面も同様に複数存在している。

http://localhost:3000/mentor/courses

image

http://localhost:3000/courses

image

機能が同じ & メンター用画面が存在するため、http://localhost:3000/courses の画面は削除したい。

期待される振る舞い

  • メンター用のメソッドはメンター用Controllerのみに定義されている
  • 機能が重複している画面が削除されている

関連Issue

#6210

@komagata komagata added this to いつかやる in 開発 May 19, 2023
@mono-nobe mono-nobe changed the title プラクティス、コース、カテゴリーのControllerの責務を集約する メンター用メソッドの責務をメンター用Controllerに集約する May 19, 2023
@mono-nobe mono-nobe changed the title メンター用メソッドの責務をメンター用Controllerに集約する メンター用Controllerメソッドをメンター用Controllerに集約する May 19, 2023
@mono-nobe mono-nobe changed the title メンター用Controllerメソッドをメンター用Controllerに集約する メンターの責務をメンター用Controllerに集約する May 19, 2023
@machida machida moved this from いつかやる to やる in 開発 May 19, 2023
@machida machida added 雑事 and removed バグ labels May 19, 2023
@github-actions
Copy link

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

@github-actions github-actions bot added the stale label Jul 19, 2023
@komagata
Copy link
Member

おそらく完了しているのでCloseします。

開発 automation moved this from やる to 完成 Jul 26, 2023
@komagata komagata removed the stale label Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
開発
  
完成
Archived in project
Development

No branches or pull requests

3 participants