管理者系のAPIはログインを必須とする - Githubissues

IkumaTadokoro / quitcost

「無職になったらいくらかかる？」を簡単に計算できるサービス

https://quitcost.herokuapp.com/

25 stars 2 forks source link

管理者系のAPIはログインを必須とする #267

Closed IkumaTadokoro closed 2 years ago

IkumaTadokoro commented 2 years ago

Closes: #266

目的

管理者画面のために用意しているAPIは、管理者でなければリクエストできないようにする。

やったこと

管理者系のAPIをAPI::Admin配下に移動（Admin共通のControllerを適用するため & RESTFULなルーティングのため）
各処理の前に管理者ログインをしているかどうかをチェックするAPI::Admin::BaseControllerを実装し、管理者系のAPIはこれを継承するようにする

PR提出前のチェックリスト:

[x] PRの関心はただ一つだけになっている & 文法的に正しく、明確かつ完全なタイトルと本文になっている
[x] 良いコミットメッセージを書いている
[x] 関連issueがある場合、コミットメッセージにClosing Keywordsを使っている
[x] Featureブランチは最新版のmainブランチに追随している (そうでなければrebaseすること)
[x] 関連するコミットはsquashした
[x] テストを追加した
[x] bin/lintとbin/rspecを実行した